Closed
Bug 885456
Opened 11 years ago
Closed 11 years ago
Schedule full-stack emulator b2g unittests on m-c, inbound and try
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: mozilla)
References
Details
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
rail
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
They've already been scheduled on cedar. It looks like crashtests, reftests and xpcshell tests are already green and can be immediately scheduled on the new build for all branches (and subsequently removed from the ics_armv7a_gecko build).
Mochitests have some issues I'm currently investigating, and Mn tests have some intermittent failures.
Aki, would you prefer I filed separate bugs per test suite or just keep track of everything here?
Assignee | ||
Comment 1•11 years ago
|
||
I'm ok tracking things here. I would like to see the results from https://tbpl.mozilla.org/?tree=Cedar&rev=1be607d22e17 so we can decide what to do with debug this first pass.
Assignee | ||
Updated•11 years ago
|
Component: Release Engineering: Automation (Release Automation) → Release Engineering: Automation (General)
QA Contact: bhearsum → catlee
Assignee | ||
Comment 2•11 years ago
|
||
This is complex enough that I need to do a builder list dump & compare.
Also, our mozilla-tests/b2g_config.py has gotten seriously ugly. Hopefully as we get rid of the old emulator builds it'll get less so.
Assignee | ||
Comment 3•11 years ago
|
||
This is the diff in builders.
This may be more far-reaching than we'd like, or it may normalize things in a way we wanted in the first place. I'm not sure I know.
Attachment #766171 -
Flags: feedback?(ahalberstadt)
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 766116 [details] [diff] [review]
(configs) full emulator test rollout, part 1
Also see the builder diff -- that's what I used to do a short eyeball verification.
Attachment #766116 -
Attachment description: (configs) [needs testing] full emulator test rollout, part 1 → (configs) full emulator test rollout, part 1
Attachment #766116 -
Flags: review?(rail)
Comment 5•11 years ago
|
||
Looking at Cedar, the right builder diff would only be adding emulator_vm crashtest-1 and -2 and -3, emulator_vm xpcshell, and emulator (without the vm) reftest-1 through -10. We can't run reftests on VMs yet, but since we can run crashtests and xpcshell on VMs, we don't have any need to run those on hardware.
Reporter | ||
Comment 6•11 years ago
|
||
Comment on attachment 766171 [details] [diff] [review]
builder list diff
Review of attachment 766171 [details] [diff] [review]:
-----------------------------------------------------------------
What philor said. I think we only want to run each suite on either a VM or physical hardware, but not both. I believe R is the only suite not capable of running on the VM's at the moment so:
C, X on b2g_emulator_vm
R on b2g_emulator
::: clean_builder
@@ -3755,4 @@
> -b2g_emulator cedar opt test crashtest-1 ScriptFactory
> -b2g_emulator cedar opt test crashtest-2 ScriptFactory
> -b2g_emulator cedar opt test crashtest-3 ScriptFactory
> -b2g_emulator cedar opt test marionette-webapi ScriptFactory
Hard to tell from the diff, but is this removing the marionette-webapi on b2g_emulator from cedar? Or will it still be running on the VM? We still need to fix some test failures for this suite on cedar.
Attachment #766171 -
Flags: feedback?(ahalberstadt) → feedback-
Comment 7•11 years ago
|
||
Comment on attachment 766116 [details] [diff] [review]
(configs) full emulator test rollout, part 1
Review of attachment 766116 [details] [diff] [review]:
-----------------------------------------------------------------
> +b2g_emulator_vm birch opt test crashtest-1 ScriptFactory
This doesn't work yet. See bug 818968.
::: mozilla-tests/b2g_config.py
@@ +279,5 @@
> 'unittest-env': {'DISPLAY': ':0'},
> 'enable_opt_unittests': True,
> 'enable_debug_unittests': False,
> 'fedora-b2g': {
> + 'opt_unittest_suites': MOCHITEST + MARIONETTE,
this change would affect all branches, including b2g18*, on which we don't use Ubuntu yet (trains...) I'm not sure if we want to switch to Ubuntu with this patch.
Attachment #766116 -
Flags: review?(rail) → review-
Assignee | ||
Comment 8•11 years ago
|
||
Ok. I was trying to do the right thing and not make our configs fuglier, but it looks like our requirements won't allow for that.
So: can someone tell me specifically what I should turn off and what I should turn on, or should I try to figure it out from the above multiple comments and most likely get it wrong again?
Reporter | ||
Comment 9•11 years ago
|
||
So the latest Cedar push made all of the crashtests and reftests (which were previously green) orange. I don't know why, but I think you should hold off on this bug for a little while.
Sorry, it is super confusing with the two emulator builds + vm/hw + opt/debug. I just wanted to switch the suites over incrementally so we could avoid bustage like what just happened. But looks like we'll need to spend some more time on them now on Cedar.
Assignee | ||
Comment 10•11 years ago
|
||
Probably due to https://bugzilla.mozilla.org/show_bug.cgi?id=885802#c5 .
Reporter | ||
Comment 11•11 years ago
|
||
(In reply to Rail Aliiev [:rail] from comment #7)
> > +b2g_emulator_vm birch opt test crashtest-1 ScriptFactory
>
> This doesn't work yet. See bug 818968.
It looks like crashtests work, it's just reftests that don't. So...
Suites to enable:
b2g_emulator_vm opt - C, X
b2g_emulator opt - R
Suites to disable:
ics_armv7a_gecko_vm opt - C, X
ics_armv7a_gecko opt - R (also C, X on cedar)
I haven't seen b2g_emulator results for debug yet, so let's leave that as is.
Reporter | ||
Comment 12•11 years ago
|
||
...and now cedar reftests/crashtests are busted for both the old and new emulator builds
Assignee | ||
Comment 13•11 years ago
|
||
Do we know what happened? It's odd to me that old emulator tests are busted, but m-c isn't.
Reporter | ||
Comment 14•11 years ago
|
||
No, I'm not sure. It looks like bug 882888 except not intermittent. I'll see if I can find some time to look at it tomorrow.
Reporter | ||
Comment 15•11 years ago
|
||
I think I fixed the OnReftestLoad problem, so when bug 882888 lands we can go ahead with this. The mochitests got fixed with bug 885454 and are also now all green. To clarify:
Step 1 - enable suites:
B2G Emu opt - R, Mn
B2G Emu (VM) opt - M, C, X
Step 2 - disable suites:
B2G Arm opt - All tests and builds!
Notes:
* Ideally we want to run Mn tests on the (VM) pool, but there are still some intermittent oranges there. If we land on the hardware pool first we'd be able to nuke the ics_armv7a_gecko builds wholesale. If you'd prefer to have them running on the (VM) pool right away we could disable the tests or look into the failures first.
* We might want to do step 1 before step 2 just to be safe, though I'll leave that up to you.
Depends on: 882888
Reporter | ||
Comment 16•11 years ago
|
||
Also just to clarify, this is only for m-c based branches. The b2g18 branches should remain as is.
Assignee | ||
Comment 17•11 years ago
|
||
Thanks Andrew!
Assignee: nobody → aki
Attachment #766116 -
Attachment is obsolete: true
Attachment #768668 -
Flags: review?(rail)
Assignee | ||
Comment 18•11 years ago
|
||
I'm guessing the cedar bit is just a re-ordering.
Attachment #766171 -
Attachment is obsolete: true
Updated•11 years ago
|
Attachment #768668 -
Flags: review?(rail) → review+
Assignee | ||
Comment 19•11 years ago
|
||
Comment on attachment 768668 [details] [diff] [review]
as requested
https://hg.mozilla.org/build/buildbot-configs/rev/82f84bdf4492
Attachment #768668 -
Flags: checked-in+
Assignee | ||
Comment 20•11 years ago
|
||
Reconfiged.
Comment 21•11 years ago
|
||
Any reason birch was left out on this? As the main integration branch for b2g, I would think running these builds and tests there would be of high importance.
Assignee | ||
Comment 22•11 years ago
|
||
Weird, I didn't special case birch at all.
Comment 23•11 years ago
|
||
I see them running now. Sorry for the false alarm.
Reporter | ||
Comment 24•11 years ago
|
||
This has been running for awhile now without problem. I'll file a new bug for turning off the old builds.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Comment 25•11 years ago
|
||
I see this in buildbot-configs:
1092 # emulator hacks. See bug 885456
1093 # MERGE DAY This will someday ride trains...
1094 for branch in BRANCHES.keys():
1095 if branch in ('mozilla-aurora', 'mozilla-beta', 'mozilla-release',
1096 'mozilla-esr17', 'mozilla-b2g18_v1_0_0',
1097 'mozilla-b2g18_v1_0_1'):
1098 if 'emulator' in BRANCHES[branch]['platforms']:
1099 del BRANCHES[branch]['platforms']['emulator']
When is the code suppose to be riding the trains?
Assignee | ||
Comment 26•11 years ago
|
||
(In reply to Armen Zambrano [:armenzg] (Release Engineering) (EDT/UTC-4) from comment #25)
> I see this in buildbot-configs:
> 1092 # emulator hacks. See bug 885456
> 1093 # MERGE DAY This will someday ride trains...
> 1094 for branch in BRANCHES.keys():
> 1095 if branch in ('mozilla-aurora', 'mozilla-beta', 'mozilla-release',
> 1096 'mozilla-esr17', 'mozilla-b2g18_v1_0_0',
> 1097 'mozilla-b2g18_v1_0_1'):
> 1098 if 'emulator' in BRANCHES[branch]['platforms']:
> 1099 del BRANCHES[branch]['platforms']['emulator']
>
> When is the code suppose to be riding the trains?
Probably now, now that b2g26 is riding the trains.
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•