Schedule Android 2.3 mochitest,xpcshell,robocop tests (only) on all trunk trees and make them ride the trains

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: gbrown, Assigned: kmoir)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

8.46 KB, patch
Armen - back on June 11th
: review+
kmoir
: checked-in+
Details | Diff | Splinter Review
23.16 KB, text/plain
Details
(Reporter)

Description

3 years ago
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

3 years ago
Assignee: nobody → kmoir
(Reporter)

Comment 1

3 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

3 years ago
Created attachment 8399453 [details] [diff] [review]
bug989462.patch

Will attach builder diff too
Attachment #8399453 - Flags: review?(armenzg)
(Assignee)

Comment 3

3 years ago
Created attachment 8399455 [details]
builder diff
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

3 years ago
Created attachment 8399549 [details] [diff] [review]
bug989462.patch
Attachment #8399453 - Attachment is obsolete: true
Attachment #8399549 - Flags: review?(armenzg)
(Assignee)

Comment 6

3 years ago
Created attachment 8399550 [details]
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+
(Assignee)

Updated

3 years ago
Attachment #8399549 - Flags: checked-in+
(Assignee)

Comment 8

3 years ago
In production and verified in tbpl
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.