Closed
Bug 499685
Opened 15 years ago
Closed 15 years ago
skip slow running acceptance tests in test step, move to deeptest
Categories
(Tamarin Graveyard :: Build Config, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: dschaffe, Assigned: dschaffe)
Details
Attachments
(5 files, 1 obsolete file)
1.89 KB,
patch
|
Details | Diff | Splinter Review | |
50.31 KB,
patch
|
brbaker
:
review+
|
Details | Diff | Splinter Review |
3.21 KB,
patch
|
brbaker
:
review+
|
Details | Diff | Splinter Review |
34.21 KB,
patch
|
brbaker
:
review+
|
Details | Diff | Splinter Review |
6.72 KB,
patch
|
brbaker
:
review+
|
Details | Diff | Splinter Review |
on sparc,x64,ppc,ppc64 a set of 10 tests on debug and debugdebugger account for > 50% of the time we should move the tests on the slower configurations to deep test phase
Flags: flashplayer-triage+
Assignee | ||
Comment 1•15 years ago
|
||
Assignee: nobody → dschaffe
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•15 years ago
|
||
Attachment #389475 -
Flags: review?(brbaker)
Updated•15 years ago
|
Attachment #389475 -
Attachment is patch: true
Attachment #389475 -
Attachment mime type: application/octet-stream → text/plain
Comment 3•15 years ago
|
||
Comment on attachment 389475 [details] [diff] [review] add ppc deep test running slow acceptance tests - Should use the "test_debug" and "test_debug_debugger" steps that are already defined - should add a "test_debug64" that could be reused when an intel mac machine is added to the deep testing - port to use in buildbot.tac is 1000 - the basedir in environment should be /Users/build/buildbot/$branch/mac-ppc-deep/$branch - rename local scripts to not contain "-ppc"
Attachment #389475 -
Flags: review?(brbaker) → review-
Assignee | ||
Comment 4•15 years ago
|
||
also I changed all buildbot.tac from port 9750 to 1000.
Attachment #389475 -
Attachment is obsolete: true
Attachment #389530 -
Flags: review?(brbaker)
Updated•15 years ago
|
Attachment #389530 -
Flags: review?(brbaker) → review+
Comment 5•15 years ago
|
||
Comment on attachment 389530 [details] [diff] [review] updated patch add deep for solaris, windows64, mac-ppc + with a couple of tweaks - builder dependencies are incorrect, should depend on the proper *-test builder and not the windows-test builder - rename the buildstep names to just be Testsuite_DebugDebugger and not contain "*64", this will make sure that the step name is matched on all builders - solaris machine path is "/home/build/...." for buildbot.tac - solaris is called "solaris-sparc" so maybe the deep should be "solaris-sparc-deep" - proper config string for solaris is "sparc-sol-tvm-debug" not "solaris-tvm-debug" - why does windows64 environment.sh set the shell_extension to nothing?
Assignee | ||
Comment 6•15 years ago
|
||
pushed in changeset 2174 f6390cd96f53, still debugging a few hg problems
Comment 7•15 years ago
|
||
Make sure the solaris-sparc-deep scripts are executable: 2203:f69378866c05
Assignee | ||
Comment 8•15 years ago
|
||
Attachment #390215 -
Flags: review?(brbaker)
Updated•15 years ago
|
Attachment #390215 -
Flags: review?(brbaker) → review+
Assignee | ||
Comment 9•15 years ago
|
||
Attachment #390467 -
Flags: review?(brbaker)
Comment 10•15 years ago
|
||
Comment on attachment 390467 [details] [diff] [review] patch to add winmo deep-test, for running slow acceptance tests + with the following quick fixes: - why are the commented out esc buildsteps in the deep builders - in the master.cfg you have the path set to slaves/winmobile-emulator-deep/scripts but the scripts are in (and have path information) for winmobile-arm-deep - *interp-arm-emulator.sh has an incorrect --config string, remove double config and you also need to include -Dinterp in the config string - wc-interp and jit scripts should have vmargs appended to the the --config string
Attachment #390467 -
Flags: review?(brbaker) → review+
Assignee | ||
Comment 11•15 years ago
|
||
thanks for feedback. on the commented out esc buildsteps, I was going to test if work on the deep test platforms before enabling esc builds. We have only run on windows thus far.
Assignee | ||
Comment 12•15 years ago
|
||
pushed patch for add winmo deep-test in revision changeset 2228 d3bc8fd964a6
Assignee | ||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•15 years ago
|
||
Attachment #390812 -
Flags: review?(brbaker)
Updated•15 years ago
|
Attachment #390812 -
Flags: review?(brbaker) → review+
Assignee | ||
Comment 14•15 years ago
|
||
closing bug by marking verified/fixed.
Status: RESOLVED → VERIFIED
Comment 15•12 years ago
|
||
changeset: 7314:325a5dd9f3c2 user: Brent Baker <brbaker@adobe.com> summary: Bug 499685: move unicode test to deep for x64-lin as the testcase is taking over 7 minutes to run on vls images http://hg.mozilla.org/tamarin-redux/rev/325a5dd9f3c2
Comment 16•12 years ago
|
||
changeset: 7315:bdc1ba72a4e4 user: Brent Baker <brbaker@adobe.com> summary: Bug 499685: move pauseForGCIfCollectionImminent test to the deep phase on MIPS as the testcase takes over 5 minutes to run http://hg.mozilla.org/tamarin-redux/rev/bdc1ba72a4e4
You need to log in
before you can comment on or make changes to this bug.
Description
•