Remove or update probes expiring in Firefox 90: BROWSER_ATTRIBUTION_ERRORS
Categories
(Firefox :: Messaging System, task, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox87 | --- | unaffected |
firefox88 | --- | unaffected |
firefox89 | + | fixed |
People
(Reporter: telemetry-probes, Assigned: andreio)
References
Details
(Whiteboard: [probe-expiry-alert])
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/plain
|
tdsmith
:
data-review+
|
Details |
The following Firefox probes will expire in the next major Firefox nightly release: version 90 [1].
BROWSER_ATTRIBUTION_ERRORS
What to do about this:
- If one, some, or all of the metrics are no longer needed, please remove them from their definitions files (Histograms.json, Scalars.yaml, Events.yaml).
- If one, some, or all of the metrics are still required, please submit a Data Collection Review [2] and patch to extend their expiry. There is a shorter form for data collection renewal [3].
If you have any problems, please ask for help on the #data-help Slack channel or the #telemetry Matrix room at https://chat.mozilla.org/#/room/#telemetry:mozilla.org. We'll give you a hand.
Your Friendly, Neighborhood Telemetry Team
[1] https://wiki.mozilla.org/Release_Management/Calendar
[2] https://wiki.mozilla.org/Firefox/Data_Collection
[3] https://github.com/mozilla/data-review/blob/master/renewal_request.md
This is an automated message sent from probe-scraper. See https://github.com/mozilla/probe-scraper for details.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 1•4 years ago
|
||
[Tracking Requested - why for this release]:
This would cause test failures in browser/components/attribution/test/browser/browser_AttributionCode* if left unchanged.
Updated•4 years ago
|
Comment 2•4 years ago
|
||
Please push the patch for this bug this week. The version number will be increased next Monday.
Assignee | ||
Comment 3•4 years ago
|
||
Hey I was looking at removing this probe but the tests seem to heavily rely on the telemetry pings to test all the failure edge cases. Would that be a good reason to keep it? Previously DS said it's not really useful.
Comment 4•4 years ago
|
||
There is new work on AMO that will result in more use of attributions, so I think it's worth holding onto this to the end of the year. I'm checking in with product on that.
Assignee | ||
Comment 5•4 years ago
|
||
I'll get a data review request to extend this, seems like the easiest fix.
Assignee | ||
Comment 6•4 years ago
|
||
Assignee | ||
Comment 7•4 years ago
|
||
Comment 8•4 years ago
|
||
Hi Andrei,
Now that I took a look at the patch that added these, my question would be whether there were any read errors associated with this ping. If no significant issue showed up, I'd just reverse the patch. However, I see that the mac support made use of this, so lets ping nalexander. In the meantime, r? for data review.
Assignee | ||
Comment 9•4 years ago
|
||
This telemetry was added because the number of enrolled users was significantly lower than expected in some experiments targeting on attribution. Unfortunately by the time the probes made it to release we shifted our focus and experimented/enrolled other types of users and I don't think we ever took a good look at the data.
I am inclined to keep it because attribution data comes up often in various experiment designs.
Assignee | ||
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Comment on attachment 9215491 [details]
data_review_1700826.txt
- Is the provided Data Collection Review complete, correct, and data-review+ by a Data Steward?
Yes.
- Is the data collection covered by the existing Firefox Privacy Notice?
Yes.
Comment 11•4 years ago
|
||
I am inclined to keep it because attribution data comes up often in various experiment designs.
Sorry to be slow clearing NI: I agree that we should extend this (and have r+ed the subsequent patch). When we want this data, we can't get it any other way.
Comment 12•4 years ago
|
||
Comment 13•4 years ago
|
||
bugherder |
Description
•