Closed Bug 1304546 (e10s-multi-aurora) Opened 8 years ago Closed 7 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: 7 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+
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: