1281190 - Keep collecting telemetry for Permissions Notifications until version 55

Status: RESOLVED FIXED

(Firefox :: General, defect, P2)

Description

[Chris H-C :chutten]

POPUP_NOTIFICATION_STATS is expiring soon [1]. We should remove it unless there's a compelling reason to keep collecting it.

[1]: https://mail.mozilla.org/pipermail/firefox-dev/2016-June/004374.html

Comment 1

[Matthew N. [:MattN]]

We definitely don't want to remove this as it's going to be used to measure the impact of the popup notification changes being worked on in the next few releases.

Comment 2

[:Paolo Amadini]

As noted in the firefox-dev post, this was opt-out but may want to extend as opt-in. Flagging for triage.

Comment 3

[:Paolo Amadini]

We'll extend all our telemetry histograms until our current target release of 55, and we'll remove the "releaseChannelCollection": "opt-out" flag from POPUP_NOTIFICATION_STATS.

Comment 4

[Mark Reid [:mreid]]

See bug 1270482 for one concrete case where we'll be using this data.

Comment 5

[Bryan Clark (DevTools PM) [:clarkbw]]

(In reply to :Paolo Amadini from comment #3)
> We'll extend all our telemetry histograms until our current target release
> of 55, and we'll remove the "releaseChannelCollection": "opt-out" flag from
> POPUP_NOTIFICATION_STATS.

For Push we expect to continue collecting and using this probe as opt-out.  Please extend the release but don't remove the opt-out; we can review this choice again after 55.  We are investigating some new experiments that will change a number of aspects of the push permission.

Comment 6

[:Paolo Amadini]

Attachment: Bug 1281190 - Keep collecting telemetry for Permissions Notifications until version 55.

Review commit: https://reviewboard.mozilla.org/r/64602/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/64602/

Comment 7

[Benjamin Smedberg]

Comment on attachment 8771449 [details]
Bug 1281190 - Keep collecting telemetry for Permissions Notifications until version 55.

https://reviewboard.mozilla.org/r/64602/#review61658

It seems weird to me that the user value we're talking about here is entirely in terms of push, but we're collecting data about all the notifications? In terms of being minimal, could we restrict this to just the push notifications and solve the same set of problems?

Comment 8

[:Paolo Amadini]

(In reply to Benjamin Smedberg AWAY UNTIL 2-AUG-2016 [:bsmedberg] from comment #7)
> It seems weird to me that the user value we're talking about here is
> entirely in terms of push, but we're collecting data about all the
> notifications? In terms of being minimal, could we restrict this to just the
> push notifications and solve the same set of problems?

Agreed. We'd need a separate opt-out histogram for push. Not sure it's worth changing at this point but Bryan, maybe the push team could do that if you need extending this again after 55? We're planning to retire the full notification histograms anyways.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/fx-team/rev/59b689bc8972
Keep collecting telemetry for Permissions Notifications until version 55. r=bsmedberg

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/59b689bc8972