Closed
Bug 989462
Opened 10 years ago
Closed 10 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)
(deleted),
patch
|
armenzg
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
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•10 years ago
|
Assignee: nobody → kmoir
![]() |
Reporter | |
Comment 1•10 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•10 years ago
|
||
Will attach builder diff too
Attachment #8399453 -
Flags: review?(armenzg)
Assignee | ||
Comment 3•10 years ago
|
||
Comment 4•10 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•10 years ago
|
||
Attachment #8399453 -
Attachment is obsolete: true
Attachment #8399549 -
Flags: review?(armenzg)
Assignee | ||
Comment 6•10 years ago
|
||
new builder diff. changes on ash are just ordering
Attachment #8399455 -
Attachment is obsolete: true
Comment 7•10 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•10 years ago
|
Attachment #8399549 -
Flags: checked-in+
Assignee | ||
Comment 8•10 years ago
|
||
In production and verified in tbpl
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•