Closed
Bug 989462
Opened 11 years ago
Closed 11 years ago
Schedule Android 2.3 mochitest,xpcshell,robocop tests (only) on all trunk trees and make them ride the trains
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gbrown, Assigned: kmoir)
References
Details
Attachments
(2 files, 2 obsolete files)
8.46 KB,
patch
|
armenzg
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
23.16 KB,
text/plain
|
Details |
Some Android 2.3 test suites are now running reliably and meet the requirements on https://wiki.mozilla.org/Sheriffing/Job_Visibility_Policy:
https://tbpl.mozilla.org/?tree=Ash&jobname=Android.*2.3.*
We should run what we can on trunk trees to avoid introducing new problems while we sort out the remaining issues (bug 967704).
At this time I am proposing that we start running
M1 ... M12 (but NOT M-gl)
rc1, rc2, rc3, rc4
X1, X2, X3
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → kmoir
Reporter | ||
Comment 1•11 years ago
|
||
In the March 29 merge to ash, X3 failed. I am looking into the problem and expect it to be resolved today.
Assignee | ||
Comment 2•11 years ago
|
||
Will attach builder diff too
Attachment #8399453 -
Flags: review?(armenzg)
Assignee | ||
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Comment on attachment 8399453 [details] [diff] [review]
bug989462.patch
Hi Kim, my apologies for the r- but I would like us to take another approach for the long term readability of mobile_config.py.
Could you please split ANDROID_2_3_MOZHARNESS_DICT into two? (enabled VS not enabled).
Also change the 'opt_unittest_suites':
1356 ANDROID_2_3_MOZHARNESS_UNITTEST_DICT = {
1357 'opt_unittest_suites': NEW_DICT_THAT_HAS_ENABLED_BY_DEFAULT_JOBS
Over here add both dictionaries:
1548 BRANCHES['ash']['platforms']['android']['vm_android_2_3'] = {
1549 'opt_unittest_suites': deepcopy(ANDROID_2_3_MOZHARNESS_DICT)
1550 }
I think this should simplify the for loop you're adding.
Attachment #8399453 -
Flags: review?(armenzg) → review-
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #8399453 -
Attachment is obsolete: true
Attachment #8399549 -
Flags: review?(armenzg)
Assignee | ||
Comment 6•11 years ago
|
||
new builder diff. changes on ash are just ordering
Attachment #8399455 -
Attachment is obsolete: true
Comment 7•11 years ago
|
||
Comment on attachment 8399549 [details] [diff] [review]
bug989462.patch
Review of attachment 8399549 [details] [diff] [review]:
-----------------------------------------------------------------
Thank you Kim!
Attachment #8399549 -
Flags: review?(armenzg) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8399549 -
Flags: checked-in+
Assignee | ||
Comment 8•11 years ago
|
||
In production and verified in tbpl
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•