1486764 - Add installTelemetryInfo to addon installed from AddonStudies

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, enhancement, P2)

Description

[Luca Greco [:rpl] [:luca] [:lgreco]]

This is a follow-up issue for Bug 1433334.

Comment 1

[Luca Greco [:rpl] [:luca] [:lgreco]]

Attachment: Bug 1486764 - Add installTelemetryInfo to addon installed from AddonStudies. r?aswan!

Comment 2

[Andrew Swan [:aswan]]

I thought I commented on this yesterday but I don't see it, why are we instrumenting these?  Normandy already does its own extensive instrumentation, seems like we should just ignore them for the purposes of telemetry about user addon installs.

Comment 3

[Luca Greco [:rpl] [:luca] [:lgreco]]

As agreed over IRC, I'm going to change this patch to mark the addon installed from the AddonStudies to a source (e.g. "internal"), and that source is going to be used in Bug 1433335 to avoid recording telemetry events.

Comment 4

[Andrew Swan [:aswan]]

Comment on attachment 9008106 [details]
Bug 1486764 - Add installTelemetryInfo to addon installed from AddonStudies. r?aswan!

Andrew Swan [:aswan] has approved the revision.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/95c51f6d719d
Add installTelemetryInfo to addon installed from AddonStudies. r=aswan

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/95c51f6d719d

Comment 7

[Luca Greco [:rpl] [:luca] [:lgreco]]

Setting qe-verify-, tested in automation.