Mention in the addonsManager.install telemetry event descriptions the source values based on the XPIProvider location names
Categories
(Toolkit :: Add-ons Manager, task, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox90 | --- | fixed |
People
(Reporter: rpl, Assigned: rpl)
References
Details
Attachments
(1 file)
As part of the informations included in the AddonsManager.install telemetry event, the source extra var can also be set to values derived from the XPIProvider's location names.
In Events.yaml we should explicitly add those values to the description of the source extra var, mainly for documentation purpose.
A list of the values that are currently missing is also mentioned in this github comment.
|
Assignee | |
Updated•3 years ago
|
|
|
Assignee | |
Comment 1•3 years ago
|
||
This patch does update the addonsManager.install telemety event description to explicitly mention other
installTelemetryInfo.source values derived from the XPIProvider location name.
Not all this values are actually expected to be used (e.g. because we don't sideload from some of those locations anymore,
and queries recently collected telemetry seems also to confirm that most of them are not part of any of the telemetry
events collected), but we still want to explicitly list them to make it easier for the Abuse Report API endpoint
(which is receiving a property derived by the same addon telemetryInto data as part of the abuse reports submitted to AMO)
to have a human readable description for each of them. See the following github issue comments for more context:
- https://github.com/mozilla/addons-server/issues/16671#issuecomment-790517567
- https://github.com/mozilla/addons-server/issues/16671#issuecomment-821112402
- https://github.com/mozilla/addons-server/issues/16671#issuecomment-823152291
- https://github.com/mozilla/addons-server/issues/16671#issuecomment-832705076
Pushed by luca.greco@alcacoop.it: https://hg.mozilla.org/integration/autoland/rev/3586f9904a85 Explicitly mention installTelemetryInfo source values derived from XPIProvider location name. r=mixedpuppy
|
||
Comment 3•3 years ago
|
||
| bugherder | ||
Description
•