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)

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) (deleted) — Splinter Review
Will attach builder diff too
Attachment #8399453 - Flags: review?(armenzg)
Attached file builder diff (obsolete) (deleted) —
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.patch (deleted) — Splinter Review
Attachment #8399453 - Attachment is obsolete: true
Attachment #8399549 - Flags: review?(armenzg)
Attached file bug989462-2.diff (deleted) —
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: 10 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: