Closed Bug 500567 Opened 15 years ago Closed 15 years ago

Throttle FF3.0.11->FF3.5rc3 major update before FF3.5 release day


(AUS Graveyard :: General, defect)

Not set


(Not tracked)



(Reporter: joduinn, Assigned: morgamic)




(3 files)

As part of FF3.5 release, we're making major updates available to FF3.0.11 users who manually check for updates. However, we need to make sure that *no* users are offered this MU as part of idle background checks.

This should be done for both beta *and* release users.
The beta and release channels are to block background checks from end users, and the two test channels to give QA a chance to make sure it's working in the production environment. Looking good on aus2-staging with this patch.

If we've not offered the MU to everyone by the time 3.0.12 comes along then I guess we'll have to bump the version in this array (just after pushing the minor update snippets).
Attachment #385299 - Flags: review?(morgamic)
Attachment #385299 - Flags: review?(morgamic) → review+
Comment on attachment 385299 [details] [diff] [review]
Throttle all 3.0.11 update channels

Looks good.
Checked in and tagged on AUS2_STAGING, AUS2_RTM_200906261046, AUS2_PRODUCTION
Note that the patch had a trailing ',' which doesn't cause a syntax error, but I've fixed that anyway.
BTW, my acceptance tests passed the config change:
Assertions: 218 right, 0 wrong, 0 ignored, 0 exceptions
prod bug in bug 500757
As discussed on IRC, the throttling check occurs before the channel may fall back. So partner builds are still seeing the MU on betatest, eg

Ben is going to get a list of active partner channels to include in the config.
Good catch, Nick - we'll have to add this and I filed a bug for AUS to handle this better via wildcards - bug 500794
This is overkill, I'm certain, but I took every fallback channel that has had more than 1 AUS ping this month except the extremely obviously fake ones. I skipped beta-cck-* but I'll grab those too, if we want them.
Embarrassingly huge list of partners to add to config.......... :X
Attachment #385484 - Flags: review?(bhearsum)
Attachment #385484 - Attachment is patch: true
Attachment #385484 - Attachment mime type: application/octet-stream → text/plain
Attachment #385484 - Flags: review?(bhearsum) → review+
Comment on attachment 385484 [details] [diff] [review]
patch for all partner channels

looks good to me
adding tomcat as he's the primary tester for this area
Fixed by association to bug 500794 -- or, it's a dupe or invalid but either way it's taken care of. <3
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.