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

VERIFIED FIXED

Status

VERIFIED FIXED
10 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

10 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

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

Comment 2

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

Updated

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

Comment 3

10 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

10 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

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

Comment 5

10 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

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

Comment 7

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

Comment 8

10 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

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

Comment 9

10 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

10 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

10 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

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

Updated

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

Comment 13

10 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

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