Closed
Bug 1393207
Opened 7 years ago
Closed 7 years ago
Mobile Firefox push to Play Store always re-sets the update rate to 10%
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(firefox56 fixed, firefox57 affected)
RESOLVED
FIXED
People
(Reporter: lizzard, Assigned: kmoir)
References
Details
Attachments
(1 file, 1 obsolete file)
1.33 KB,
patch
|
jcristau
:
review+
jcristau
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
When we push the APKs to the Play Store for beta releases, the update rate is always reset to 10%. We have to adjust it by hand for every beta release. Intentional, or something we can fix for the future?
Assignee | ||
Comment 1•7 years ago
|
||
My understanding from mtabara that it is intentional to allow relman to adjust the rate as releng doesn't have access to the store. Also, the default is at a low rate to allow a staged rollout as needed. Is there a better default rate that you would like us to set it to in automation?
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(lhenry)
Reporter | ||
Comment 2•7 years ago
|
||
Let's check with Sylvestre here for what he intends. (No rush, i think he is on PTO) Is there a way to leave it at whatever the previous push was rather than re-setting it with each beta?
Flags: needinfo?(lhenry) → needinfo?(sledru)
Comment 3•7 years ago
|
||
My intent in bug 1358420 is for the first beta to use 10%, and then later on to either adjust the percentage or just backout that change on m-b when we were ready to go 100%, so subsequent betas wouldn't need manual adjustment. I'm afraid I didn't actually communicate or document that though, my bad...
Comment 4•7 years ago
|
||
e.g. https://hg.mozilla.org/releases/mozilla-beta/rev/544e640571f4
Assignee | ||
Comment 5•7 years ago
|
||
I'll also open a bug to reset the rate to 10% before the next b1.
Assignee: nobody → kmoir
Attachment #8900727 -
Flags: review?(jcristau)
Comment 6•7 years ago
|
||
(In reply to Kim Moir [:kmoir] ET from comment #5) > I'll also open a bug to reset the rate to 10% before the next b1. That shouldn't be necessary, m-c already has the 10% rate, and any change we make for 56 should only land on m-b.
Comment 7•7 years ago
|
||
Comment on attachment 8900727 [details] [diff] [review] bug1393207.patch Review of attachment 8900727 [details] [diff] [review]: ----------------------------------------------------------------- better backout bug 1358420 on m-b entirely imo if we're going 100%
Attachment #8900727 -
Flags: review?(jcristau) → review-
Assignee | ||
Comment 9•7 years ago
|
||
Attachment #8900727 -
Attachment is obsolete: true
Attachment #8900739 -
Flags: review?(jcristau)
Comment 10•7 years ago
|
||
Comment on attachment 8900739 [details] [diff] [review] bug1393207-2.patch Review of attachment 8900739 [details] [diff] [review]: ----------------------------------------------------------------- lgtm
Attachment #8900739 -
Flags: review?(jcristau)
Attachment #8900739 -
Flags: review+
Attachment #8900739 -
Flags: approval-mozilla-beta+
Comment 11•7 years ago
|
||
uplift |
Kim pushed this to Beta. I assume it'll be landing on inbound still. https://hg.mozilla.org/releases/mozilla-beta/rev/aa85d946916f
status-firefox56:
--- → fixed
status-firefox57:
--- → affected
Assignee | ||
Comment 12•7 years ago
|
||
No, I didn't land it on inbound because we want it to be reverted for the next b1. See comment 6
Assignee | ||
Comment 14•7 years ago
|
||
Was the google play rate set to 100% with Fennec b7? Please verify so I can close this bug? Also, a fyi when we do the merge and the first 57.0b1 runs, the build will run the rollout plan again at 10% since we just landed this change on beta, not on trunk. So the google play store config for beta will have be changed again for the rollout plan instead of the full release
Flags: needinfo?(lhenry)
Assignee | ||
Comment 15•7 years ago
|
||
from #releaseduty jcristau> Julien Cristau kmoir: Rollout 100% for fennec beta 56.0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 16•7 years ago
|
||
OK. Thanks Kim. We will keep it in mind to check on the update rate and adjust it after beta 1 rollout.
Flags: needinfo?(lhenry)
You need to log in
before you can comment on or make changes to this bug.
Description
•