Closed Bug 1182691 Opened 7 years ago Closed 7 years ago

Run Android 2.3 and Android 4.3 mochitest-chrome on trunk

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(firefox42 fixed)

RESOLVED FIXED
Tracking Status
firefox42 --- fixed

People

(Reporter: gbrown, Assigned: kmoir)

References

Details

Attachments

(4 files, 4 obsolete files)

We can start running mochitest-chrome on Android 2.3 API9 opt, Android 4.3 API11+ opt, and Android 4.3 API11+ debug on all trunk trees.

Android mochitest-chrome is just like mochitest but --chrome is added to the runtestsremote.py arguments, and we can run all mochitest-chrome in a single chunk.

I think this should display in treeherder as M(c) (like b2g mochitest-chrome). M(oth) is another option (desktop includes mochitest-chrome in "other")...but we have no plans to run additional mochitest flavors on Android.
Attached patch (wip) mozharness config update (obsolete) — Splinter Review
Attached patch bug1182691-mh (obsolete) — Splinter Review
I know jlund is in the middle of moving mozharness in-tree; I will check with him before pushing.

kmoir -- Will you be able to write the buildbot changes to trigger this new job?
Assignee: nobody → gbrown
Attachment #8632344 - Attachment is obsolete: true
Attachment #8632837 - Flags: review?(kmoir)
Currently --total-chunks is defined in both the mozharness config and the in-tree config. I'm removing it from the in-tree config because that applies to all mochitests, and now mochitest-chrome shares the mochitests config.
Attachment #8632340 - Attachment is obsolete: true
Attachment #8632841 - Flags: review?(kmoir)
Sorry -- attached wrong patch!
Attachment #8632837 - Attachment is obsolete: true
Attachment #8632837 - Flags: review?(kmoir)
Attachment #8632843 - Flags: review?(kmoir)
Attachment #8632841 - Flags: review?(kmoir) → review+
Comment on attachment 8632843 [details] [diff] [review]
add mochitest-chrome to mozharness configs

Yes, I can write the buildbot config changes for this
Attachment #8632843 - Flags: review?(kmoir) → review+
Attached patch bug1182691.patch (obsolete) — Splinter Review
buildbot configs
Attached file bug1182691builder.diff
builder diffs
initially attached wrong patch.  

The loadSkipConfig(BRANCHES,"mobile") had to be moved due to an ordering issue
Attachment #8633032 - Attachment is obsolete: true
Attachment #8633074 - Flags: review?(jlund)
Assignee: gbrown → kmoir
https://hg.mozilla.org/mozilla-central/rev/a537c28456f4
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Should have mentioned when I attached the patch that I verified that watch_pending.cfg doesn't need to be modified since it will just fall under mochitest
Sorry, forgot to leave-open.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment on attachment 8633074 [details] [diff] [review]
bug1182691-2.patch

Review of attachment 8633074 [details] [diff] [review]:
-----------------------------------------------------------------

that's too bad about loadSkipConfig. Looks good to me anyway :)

::: mozilla-tests/mobile_config.py
@@ +2580,5 @@
>  # schedule instrumentation tests for pandas on ash and cedar
>  # https://bugzilla.mozilla.org/show_bug.cgi?id=1064010
>  remove_suite_from_slave_platform(BRANCHES, PLATFORMS, 'instrumentation', 'panda_android', branches_to_keep=['cedar'])
>  
> +# Bug 1182691 - Run Android 2.3 and Android 4.3 mochitest-chrome on trunk 

white space
Attachment #8633074 - Flags: review?(jlund) → review+
Comment on attachment 8633074 [details] [diff] [review]
bug1182691-2.patch

fixed whitespace too
Attachment #8633074 - Flags: checked-in+
verified on treeherder tests are green on m-c
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.