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)

x86_64
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gbrown, Assigned: kmoir)

References

Details

Attachments

(2 files, 2 obsolete files)

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: nobody → kmoir
In the March 29 merge to ash, X3 failed. I am looking into the problem and expect it to be resolved today.
Attached patch bug989462.patch (obsolete) — Splinter Review
Will attach builder diff too
Attachment #8399453 - Flags: review?(armenzg)
Attached file builder diff (obsolete) —
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-
Attached patch bug989462.patchSplinter Review
Attachment #8399453 - Attachment is obsolete: true
Attachment #8399549 - Flags: review?(armenzg)
Attached file bug989462-2.diff
new builder diff. changes on ash are just ordering
Attachment #8399455 - Attachment is obsolete: true
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+
Attachment #8399549 - Flags: checked-in+
In production and verified in tbpl
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: