Closed
Bug 1219342
Opened 9 years ago
Closed 9 years ago
Enable e10s tests on mozilla-beta when Gecko 43 is uplifted to it
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(e10s+)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
e10s | + | --- |
People
(Reporter: elan, Unassigned)
References
Details
Attachments
(1 file)
2.24 KB,
patch
|
Details | Diff | Splinter Review |
e10s tests are enabled for Nightly and Aurora and now we need them on Beta.
Updated•9 years ago
|
Blocks: e10s-beta
Component: General → General Automation
Product: Firefox → Release Engineering
QA Contact: catlee
Summary: Enable Tests on Firefox 43.0 Beta → Enable e10s tests on mozilla-beta when Gecko 43 is uplifted to it
Version: 43 Branch → ---
![]() |
||
Comment 1•9 years ago
|
||
We need to figure out what needs to happen here, specifically:
1) what suites are we turning on
2) patches for turning them on
3) we should run beta43 through try + e10s test before we do this in production
related bug: Bug 1218475 - Opt-In Enable e10s for Firefox 43.0 Beta
Blocks: 1218475
tracking-e10s:
--- → ?
Comment 2•9 years ago
|
||
My main concern here is that prior experience with uplift simulation Try pushes (which mostly end up affecting the NIGHTLY_BUILD/RELEASE_BUILD ifdefs) is that e10s tests do not respond well to attempting to run on channels they aren't meant to be.
I'm worried that if we enable these tests, we're going to have a pile of orange waiting for us since nothing is changing from an ifdef standpoint.
Comment 3•9 years ago
|
||
This patch should do the trick from the buildbot side of things. A builder diff shows the expected results.
As Jim said, this can't land until we've verified that things are green on Try. My expectation is that currently, they won't be. After tomorrow's uplift, pushing m-b to Try directly should suffice for testing, however.
Updated•9 years ago
|
Comment 4•9 years ago
|
||
bug 1226487 enabled for A/B beta test.
Noticed app.update.channel is default on basic treeheader beta build, i.e. e10s disabled. Something probably needs changing when this bug is dealt with.
Comment 5•9 years ago
|
||
Ryan, do we still need to uplift e10s tests to mozilla-beta? Is this bug still relevant?
Flags: needinfo?(ryanvm)
Comment 6•9 years ago
|
||
No, this is long-done.
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(ryanvm)
Resolution: --- → FIXED
Assignee | ||
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
•