Closed
Bug 499685
Opened 16 years ago
Closed 16 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•16 years ago
|
||
Assignee: nobody → dschaffe
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•16 years ago
|
||
Attachment #389475 -
Flags: review?(brbaker)
Updated•16 years ago
|
Attachment #389475 -
Attachment is patch: true
Attachment #389475 -
Attachment mime type: application/octet-stream → text/plain
Comment 3•16 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•16 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•16 years ago
|
Attachment #389530 -
Flags: review?(brbaker) → review+
Comment 5•16 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•16 years ago
|
||
pushed in changeset 2174 f6390cd96f53, still debugging a few hg problems
Comment 7•16 years ago
|
||
Make sure the solaris-sparc-deep scripts are executable: 2203:f69378866c05
Assignee | ||
Comment 8•16 years ago
|
||
Attachment #390215 -
Flags: review?(brbaker)
Updated•16 years ago
|
Attachment #390215 -
Flags: review?(brbaker) → review+
Assignee | ||
Comment 9•16 years ago
|
||
Attachment #390467 -
Flags: review?(brbaker)
Comment 10•16 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•16 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•16 years ago
|
||
pushed patch for add winmo deep-test in revision changeset 2228 d3bc8fd964a6
Assignee | ||
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•16 years ago
|
||
Attachment #390812 -
Flags: review?(brbaker)
Updated•16 years ago
|
Attachment #390812 -
Flags: review?(brbaker) → review+
Assignee | ||
Comment 14•16 years ago
|
||
closing bug by marking verified/fixed.
Status: RESOLVED → VERIFIED
Comment 15•13 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•13 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
•