Closed Bug 1257181 Opened 4 years ago Closed 4 years ago
Re-enable APZ on beta
+++ This bug was initially created as a clone of Bug #1253667 +++ It was decided that, after doing the first few 46 beta builds with APZ off, we would turn it back on in beta4 for a couple of betas, and then the e10s experiment in 46 beta would end. So basically this bug is to back out bug 1253667 and land it on beta for 46.0b4. See bug 1253667 comment 11 as well.
4 years ago
Approval Request Comment [Feature/regressing bug #]: APZ [User impact if declined]: APZ won't be enabled in beta for the e10s+APZ experiment [Describe test coverage new/current, TreeHerder]: all the APZ stuff [Risks and why]: This was discussed in the e10s cross-functional [String/UUID change made/needed]: none
kats, just to be clear, we plan to disable e10s on Beta 46 after the e10s+APZ experiment ends, which will implicitly disable APZ (regardless of the value of the APZ pref). Is that still the plan?
Yes, I agree with that.
4 years ago
Comment on attachment 8731250 [details] [diff] [review] Patch OK to turn on apz by default for aurora and beta. This should make it into beta 4.
Comment on attachment 8731250 [details] [diff] [review] Patch Review of attachment 8731250 [details] [diff] [review]: ----------------------------------------------------------------- please hold off landing on beta until we get agreement at the meeting tomorrow, and also because I want to make sure this only gets into the right beta and not earlier. If you want to land on aurora sooner, go ahead.
Attachment #8731250 - Flags: review?(felipc) → review+
I'll land this on inbound/aurora/beta once bug 1247854 and it's dependencies is safely on beta. I'll be around this weekend and can do it then if needed.
I verified bug 1247854 is fixed on the build that has the uplift, so I landed the APZ-enabling patch: https://hg.mozilla.org/releases/mozilla-aurora/rev/6d030cb1d2ab https://hg.mozilla.org/releases/mozilla-beta/rev/02bdd0469632 Looks like the tag for FIREFOX_46_0b2_BUILD3, FIREFOX_46_0b2_RELEASE already happened so this will be in beta 3 as discussed during the meeting. I hope that's the right beta, not sure if plans are changing because of the updates being turned off.
oh no, apz and the regressions: https://treeherder.mozilla.org/perf.html#/alerts?id=566 and a compare view: https://treeherder.mozilla.org/perf.html#/compare?originalProject=mozilla-beta&originalRevision=24b6aaac8973&newProject=mozilla-beta&newRevision=02bdd0469632&framework=1 these are all e10s regressions, that is even scarier!
:kats, i assume this was expected? please confirm, or outline the next steps.
Yeah, this is expected. APZ is not riding to release on 46; we are tracking the regressions on Nightly and fixing them there.
You need to log in before you can comment on or make changes to this bug.