Closed Bug 1335193 Opened 3 years ago Closed 3 years ago

Switch the alert emails for scroll tracking histograms

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox52 --- fixed
firefox-esr52 --- fixed
firefox53 --- fixed
firefox54 --- fixed

People

(Reporter: rhunt, Assigned: rhunt)

Details

Attachments

(1 file)

Harald can pay attention to the alert emails better than I can.
Attachment #8831815 - Flags: review?(hkirschner)
Comment on attachment 8831815 [details] [diff] [review]
alert-emails.patch

r+
Attachment #8831815 - Flags: review?(hkirschner) → review+
Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6ef293b9ca05
Switch the alert emails for scroll tracking histograms. r=digitarald
https://hg.mozilla.org/mozilla-central/rev/6ef293b9ca05
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Comment on attachment 8831815 [details] [diff] [review]
alert-emails.patch

Approval Request Comment
[Feature/Bug causing the regression]: These scroll tracking patches have been uplifted to 52, but the alert-email for them is still the old one.
[User impact if declined]: None on user, this is just for telemetry
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: It just changes who gets the emails for a telemetry probe
[String changes made/needed]: None
Attachment #8831815 - Flags: approval-mozilla-beta?
Comment on attachment 8831815 [details] [diff] [review]
alert-emails.patch

update alert email for a couple of telemetry probes, beta52+

Ryan, this should probably go in 53 (aurora) as well?
Flags: needinfo?(rhunt)
Attachment #8831815 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Ah yes, this probably should go in 53 too. This patch missed the merge date for that.
Flags: needinfo?(rhunt)
Comment on attachment 8831815 [details] [diff] [review]
alert-emails.patch

Approval Request Comment
[Feature/Bug causing the regression]: These scroll tracking patches have been uplifted to 52, but the alert-email for them is still the old one.
[User impact if declined]: None on user, this is just for telemetry
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: It just changes who gets the emails for a telemetry probe
[String changes made/needed]: None
Attachment #8831815 - Flags: approval-mozilla-aurora?
Comment on attachment 8831815 [details] [diff] [review]
alert-emails.patch

get this in aurora so we're not regressing from 52 to 53.
Attachment #8831815 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.