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)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox52 --- wontfix
firefox53 --- wontfix
firefox54 --- fixed
firefox55 --- fixed

People

(Reporter: mrbkap, Assigned: gkrizsanits)

References

(Depends on 1 open bug)

Details

(Keywords: meta)

Attachments

(1 file)

This is a placeholder bug to turn e10s-multi on (for some number of processes to be determined at a later date) in Aurora.
Depends on: 1307511
Depends on: 1289723
Depends on: 1324428
Depends on: 1284201
Depends on: 1285888
Depends on: 1272113
Depends on: 1306458
Depends on: 1315042
Alias: e10s-multi-aurora
Depends on: 1333798
Depends on: 1333799
Depends on: 1336389
Depends on: 1336398
Depends on: 1337778
Depends on: 1340512
Depends on: 1334716
Depends on: 1290167
Depends on: 1301015
Depends on: 1341008
Depends on: 1208957
No longer depends on: 1290167
Depends on: 1325049
Depends on: 1352706
Assignee: nobody → gkrizsanits
Attachment #8854045 - Flags: review?(mrbkap)
Attachment #8854045 - Flags: review?(mrbkap) → review+
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?
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
https://hg.mozilla.org/mozilla-central/rev/5a0fce48c63f
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Tweaking the status flags so we won't lose the uplift request
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+
https://hg.mozilla.org/releases/mozilla-aurora/rev/29eab893bad0
(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.
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)
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)
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)
(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)
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)
Depends on: 1353910
No longer depends on: 1352706
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: