Closed
Bug 1182691
Opened 9 years ago
Closed 9 years ago
Run Android 2.3 and Android 4.3 mochitest-chrome on trunk
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(firefox42 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: gbrown, Assigned: kmoir)
References
Details
Attachments
(4 files, 4 obsolete files)
2.65 KB,
patch
|
kmoir
:
review+
|
Details | Diff | Splinter Review |
1.87 KB,
patch
|
kmoir
:
review+
|
Details | Diff | Splinter Review |
2.40 KB,
text/plain
|
Details | |
3.92 KB,
patch
|
jlund
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
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.
![]() |
Reporter | |
Comment 1•9 years ago
|
||
![]() |
Reporter | |
Comment 2•9 years ago
|
||
![]() |
Reporter | |
Comment 3•9 years ago
|
||
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)
![]() |
Reporter | |
Comment 4•9 years ago
|
||
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
![]() |
Reporter | |
Updated•9 years ago
|
Attachment #8632841 -
Flags: review?(kmoir)
![]() |
Reporter | |
Comment 5•9 years ago
|
||
Sorry -- attached wrong patch!
Attachment #8632837 -
Attachment is obsolete: true
Attachment #8632837 -
Flags: review?(kmoir)
Attachment #8632843 -
Flags: review?(kmoir)
![]() |
Reporter | |
Comment 6•9 years ago
|
||
Verified on try at https://treeherder.mozilla.org/#/jobs?repo=try&revision=13a2ee74657e with this hack to simulate buildbot: http://hg.mozilla.org/users/gbrown_mozilla.com/gbmozharness/rev/7de57f189f1c
Assignee | ||
Updated•9 years ago
|
Attachment #8632841 -
Flags: review?(kmoir) → review+
Assignee | ||
Comment 7•9 years ago
|
||
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+
![]() |
Reporter | |
Comment 8•9 years ago
|
||
https://hg.mozilla.org/build/mozharness/rev/4addb10922bc
Assignee | ||
Comment 9•9 years ago
|
||
buildbot configs
Assignee | ||
Comment 10•9 years ago
|
||
builder diffs
Assignee | ||
Comment 11•9 years ago
|
||
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)
![]() |
Reporter | |
Updated•9 years ago
|
Assignee: gbrown → kmoir
Assignee | ||
Comment 14•9 years ago
|
||
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
![]() |
Reporter | |
Comment 15•9 years ago
|
||
Sorry, forgot to leave-open.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 16•9 years ago
|
||
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+
Assignee | ||
Comment 17•9 years ago
|
||
Comment on attachment 8633074 [details] [diff] [review] bug1182691-2.patch fixed whitespace too
Attachment #8633074 -
Flags: checked-in+
Comment 19•9 years ago
|
||
In production: https://hg.mozilla.org/build/buildbot-configs/rev/fa0bbb6e7b44
Comment 20•9 years ago
|
||
In production: https://hg.mozilla.org/build/buildbot-configs/rev/fa0bbb6e7b44
Assignee | ||
Comment 21•9 years ago
|
||
verified on treeherder tests are green on m-c
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 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
•