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

Status: RESOLVED FIXED

(AUS Graveyard :: General, defect)

Description

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

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.

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: Throttle all 3.0.11 update channels

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).

Comment 2

[Michael Morgan [:morgamic]]

Comment on attachment 385299 [details] [diff] [review]
Throttle all 3.0.11 update channels

Looks good.

Comment 3

[Michael Morgan [:morgamic]]

Checked in and tagged on AUS2_STAGING, AUS2_RTM_200906261046, AUS2_PRODUCTION

Comment 4

[Michael Morgan [:morgamic]]

Note that the patch had a trailing ',' which doesn't cause a syntax error, but I've fixed that anyway.

Comment 5

[Michael Morgan [:morgamic]]

BTW, my acceptance tests passed the config change:
Assertions: 218 right, 0 wrong, 0 ignored, 0 exceptions

Comment 6

[Michael Morgan [:morgamic]]

prod bug in bug 500757

Comment 7

[Nick Thomas [:nthomas] (UTC+12)]

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
https://aus2.mozilla.org/update/3/Firefox/3.0.11/2009060214/Darwin_Universal-gcc3/en-US/betatest-cck-kodakgallery/Darwin%208.11.1/kodakgallery/2.0/update.xml

Ben is going to get a list of active partner channels to include in the config.

Comment 8

[Michael Morgan [:morgamic]]

Good catch, Nick - we'll have to add this and I filed a bug for AUS to handle this better via wildcards - bug 500794

Comment 9

[[e-mail] (:bhearsum)]

Attachment: list of partner fallback channels

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.

Comment 10

[Michael Morgan [:morgamic]]

Attachment: patch for all partner channels

Embarrassingly huge list of partners to add to config.......... :X

Comment 11

[[e-mail] (:bhearsum)]

Comment on attachment 385484 [details] [diff] [review]
patch for all partner channels

looks good to me

Comment 12

[Tony Chung [:tchung]]

adding tomcat as he's the primary tester for this area

Comment 13

[Michael Morgan [:morgamic]]

Fixed by association to bug 500794 -- or, it's a dupe or invalid but either way it's taken care of. <3