1492356 - ship Telemetry Coverage SAO update to Firefox 45 through 60

Status: RESOLVED FIXED

(Data Platform and Tools :: General, enhancement)

Description

[Robert Helmer [:rhelmer]]

In bug 1487578 we shipped the Telemetry Coverage system add-on update to Firefox 61 and 62 (the latest two releases). We'd like to ship this to Firefox 45 through 60, but the extension needs some changes to do so.

I've prepared a version of the extension from bug 1487578 that works on both 45 and also current Nightly (64), and (hopefully) everything in between:

https://github.com/mozilla/one-off-system-add-ons/pull/131

Comment 1

[Robert Helmer [:rhelmer]]

Attachment: Telemetry Coverage (8.0, compatible with Firefox 45+)

Please sign this system add-on update, thanks!

Comment 2

[Robert Helmer [:rhelmer]]

Rehan, when the signed XPI is ready, is it possible serve this on the release-sysaddon channel to all users on Firefox 45 through 60?

Comment 3

[:wezhou]

Attachment: signed.9010139.xpi

Signed file attached. Please test.

Comment 4

[Rehan Dalal (Inactive)]

Redirecting to mythmon since he will be handling this while I'm away on PTO

Comment 5

[Michael Cooper [:mythmon]]

I've added the system add-on to all rules on the release-sysaddon channel targeting versions of Firefox 45-60, inclusive.

Comment 6

[Robert Helmer [:rhelmer]]

(In reply to Michael Cooper [:mythmon] from comment #5)
> I've added the system add-on to all rules on the release-sysaddon channel
> targeting versions of Firefox 45-60, inclusive.

Thanks! I will try it out on a few releases.

Ritu, do you want us to do formal QA? This is shipping to unsupported releases.

Comment 7

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

From a risk assessment point of view, shipping to unsupported versions has minimal risk associated. Rob, have you tested whether this works on an old (< 61) version of Firefox?

Also NI TomGrab, as QA lead, if he wants to enforce a QA sign off

Comment 8

[Tom Grabowski [:TomGrab]]

We don't need formal QA sign-off for unsupported versions. But I still expect that developers will conduct some basic testing and share their results here.

Comment 9

[Robert Helmer [:rhelmer]]

Thanks for the quick confirmations!

I've tested that this works on Firefox 45 and 60, and am looking at Telemetry to make sure the packets were received and processed correctly before we push this.

Comment 10

[Robert Helmer [:rhelmer]]

OK I see results in Telemetry so I think we're good to go here. I've tested on 45 and 60.

:mythmon, I think this is ready to go live on release channel for 45-60 only.

(Technically this works fine on 61/62 in my testing, but I'd rather leave 61/62 alone since we did a ton of QA etc. on the exact version that went out already.)

Comment 11

[Michael Cooper [:mythmon]]

I have queued changes for the release channel for Firefox Release versions 45-60, inclusive. Ritu, can you please approve the rules?

Comment 12

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Done. Sorry for the delay on this one.

Comment 13

[Mark Reid [:mreid]]

Rob, is the "leave-open" tag still needed? Can we close this bug out?

Comment 14

[Robert Helmer [:rhelmer]]

(In reply to Mark Reid [:mreid] from comment #13)
> Rob, is the "leave-open" tag still needed? Can we close this bug out?

I think we're done here. Thanks!