Closed
Bug 1304546
(e10s-multi-aurora)
Opened 8 years ago
Closed 8 years ago
[meta] Turn e10s-multi on in Aurora-DevEdition
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
FIXED
mozilla55
People
(Reporter: mrbkap, Assigned: gkrizsanits)
References
(Depends on 1 open bug)
Details
(Keywords: meta)
Attachments
(1 file)
928 bytes,
patch
|
mrbkap
:
review+
gchang
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
This is a placeholder bug to turn e10s-multi on (for some number of processes to be determined at a later date) in Aurora.
Reporter | ||
Updated•8 years ago
|
Blocks: e10s-multi-beta
Depends on: 1333914
Reporter | ||
Updated•8 years ago
|
Depends on: ServiceWorkers-e10s
Reporter | ||
Updated•8 years ago
|
Alias: e10s-multi-aurora
Assignee | ||
Comment 1•8 years ago
|
||
Assignee: nobody → gkrizsanits
Attachment #8854045 -
Flags: review?(mrbkap)
Reporter | ||
Updated•8 years ago
|
Attachment #8854045 -
Flags: review?(mrbkap) → review+
Reporter | ||
Comment 2•8 years ago
|
||
Comment on attachment 8854045 [details] [diff] [review] Enable 4 content processes on aurora Approval Request Comment [Feature/Bug causing the regression]: e10s-multi [User impact if declined]: n/a [Is this code covered by automated tests?]: Yes. [Has the fix been verified in Nightly?]: Yes. [Needs manual test from QE? If yes, steps to reproduce]: No. [List of other uplifts needed for the feature/fix]: We're uplifting as we go.
Attachment #8854045 -
Flags: approval-mozilla-aurora?
Reporter | ||
Comment 3•8 years ago
|
||
Note to sheriffs: this change has no effect on Nightly (we've already enabled e10s-multi on Nightly).
Pushed by gkrizsanits@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/5a0fce48c63f Turning on e10s-multi in Aurora. r=mrbkap
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5a0fce48c63f
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Comment 6•8 years ago
|
||
Tweaking the status flags so we won't lose the uplift request
status-firefox53:
--- → wontfix
status-firefox54:
--- → affected
Comment 7•8 years ago
|
||
Comment on attachment 8854045 [details] [diff] [review] Enable 4 content processes on aurora Enable e10s-multi on aurora and see how it goes. Aurora54+.
Attachment #8854045 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 8•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/29eab893bad0
Comment 9•8 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #8) > https://hg.mozilla.org/releases/mozilla-aurora/rev/29eab893bad0 it seems this is causing leaks like https://treeherder.mozilla.org/logviewer.html#?job_id=88700020&repo=mozilla-aurora i did some bisecting in https://hg.mozilla.org/try/rev/110f708df669971abf55dbab23056701e2fbb380 and that try run is entirely green.
Comment 10•8 years ago
|
||
gabor, blake: since this is a very important change i guess i we either need to fix or back this change out from aurora
Flags: needinfo?(mrbkap)
Flags: needinfo?(gkrizsanits)
Assignee | ||
Comment 11•8 years ago
|
||
I'm going to try to fix this in the test, but I wonder why this might be happening only on aurora and not on central... Sotaro, I think you worked on a similar issue before in bug 1285238, do you know if there is any diversion between aurora and nightly that can cause this leak? (In reply to Carsten Book [:Tomcat] from comment #9) > (In reply to Ryan VanderMeulen [:RyanVM] from comment #8) > > https://hg.mozilla.org/releases/mozilla-aurora/rev/29eab893bad0 > > it seems this is causing leaks like > https://treeherder.mozilla.org/logviewer.html#?job_id=88700020&repo=mozilla- > aurora > > i did some bisecting in > https://hg.mozilla.org/try/rev/110f708df669971abf55dbab23056701e2fbb380 and > that try run is entirely green.
Flags: needinfo?(gkrizsanits) → needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 12•8 years ago
|
||
RyanVM|mtg> gabor, mrbkap: 3:21 PM https://treeherder.mozilla.org/logviewer.html#?job_id=88717085&repo=mozilla-aurora is also happening frequently on Windows and it wouldn't shock me if it turned out to be the same root issue DΓ£o, are you aware of any diversion between aurora and nightly that could cause this failure?
Flags: needinfo?(dao+bmo)
Comment 13•8 years ago
|
||
(In reply to Gabor Krizsanits [:krizsa :gabor] from comment #12) > RyanVM|mtg> gabor, mrbkap: > 3:21 PM > https://treeherder.mozilla.org/logviewer.html#?job_id=88717085&repo=mozilla- > aurora is also happening frequently on Windows and it wouldn't shock me if > it turned out to be the same root issue > > DΓ£o, are you aware of any diversion between aurora and nightly that could > cause this failure? Nope... redirecting to Florian who wrote this test and the feature being tested.
Flags: needinfo?(dao+bmo) → needinfo?(florian)
Comment 14•8 years ago
|
||
Let's move the leak discussion over to bug 1347625, which I believe is the same root cause. Regarding the Windows failures in browser_devices_get_user_media_anim.js, I believe Gabor is still investigating.
Flags: needinfo?(sotaro.ikeda.g)
Flags: needinfo?(mrbkap)
Flags: needinfo?(florian)
You need to log in
before you can comment on or make changes to this bug.
Description
•