skip slow running acceptance tests in test step, move to deeptest

VERIFIED FIXED

Status

VERIFIED FIXED
9 years ago
7 years ago

People

(Reporter: dschaffe, Assigned: dschaffe)

Tracking

unspecified
x86
Mac OS X
Bug Flags:
flashplayer-triage +

Details

Attachments

(5 attachments, 1 obsolete attachment)

(Assignee)

Description

9 years ago
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

9 years ago
Created attachment 384454 [details] [diff] [review]
patch
Assignee: nobody → dschaffe
Status: NEW → ASSIGNED
(Assignee)

Comment 2

9 years ago
Created attachment 389475 [details] [diff] [review]
add ppc deep test running slow acceptance tests
Attachment #389475 - Flags: review?(brbaker)

Updated

9 years ago
Attachment #389475 - Attachment is patch: true
Attachment #389475 - Attachment mime type: application/octet-stream → text/plain

Comment 3

9 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

9 years ago
Created attachment 389530 [details] [diff] [review]
updated patch add deep for solaris, windows64, mac-ppc

also I changed all buildbot.tac from port 9750 to 1000.
Attachment #389475 - Attachment is obsolete: true
Attachment #389530 - Flags: review?(brbaker)

Updated

9 years ago
Attachment #389530 - Flags: review?(brbaker) → review+

Comment 5

9 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

9 years ago
pushed in changeset 2174 f6390cd96f53, still debugging a few hg problems

Comment 7

9 years ago
Make sure the solaris-sparc-deep scripts are executable: 2203:f69378866c05
(Assignee)

Comment 8

9 years ago
Created attachment 390215 [details] [diff] [review]
add util_process_clean to deep, skip mops on ppc-mac-debug, +x solaris scripts
Attachment #390215 - Flags: review?(brbaker)

Updated

9 years ago
Attachment #390215 - Flags: review?(brbaker) → review+
(Assignee)

Comment 9

9 years ago
Created attachment 390467 [details] [diff] [review]
patch to add winmo deep-test, for running slow acceptance tests
Attachment #390467 - Flags: review?(brbaker)

Comment 10

9 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

9 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

9 years ago
pushed patch for add winmo deep-test in revision changeset 2228 d3bc8fd964a6
(Assignee)

Updated

9 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Comment 13

9 years ago
Created attachment 390812 [details] [diff] [review]
patch : deep tests are included and not matched by skip patterns
Attachment #390812 - Flags: review?(brbaker)

Updated

9 years ago
Attachment #390812 - Flags: review?(brbaker) → review+
(Assignee)

Comment 14

9 years ago
closing bug by marking verified/fixed.
Status: RESOLVED → VERIFIED

Comment 15

7 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

7 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.