Data review for Firefox Monitor as an in-tree system add-on
Categories
(Firefox :: Firefox Monitor, enhancement)
Tracking
()
People
(Reporter: nhnt11, Assigned: nhnt11)
References
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
Once the add-on lands in tree (with Telemetry disabled) we should get it data-reviewed again.
Assignee | ||
Comment 1•4 years ago
•
|
||
- What questions will you answer with this data?
- How often does user encounter the feature?
- How does user respond to the feature?
- Why does Mozilla need to answer these questions? Are there benefits for users? Do we need this information to address product or business requirements?
- Provide information essential for advancing a business objective such as supporting OKRs.
- Determine whether a product or platform change has an effect on user or browser behavior.
- What alternative methods did you consider to answer these questions? Why were they not sufficient?
- There are no alternatives other than the event telemetry to learn how user interacts with this UI.
- Can current instrumentation answer these questions?
- No.
- List all proposed measurements and indicate the category of data collection for each measurement, using the Firefox data collection categories on the Mozilla wiki.
- Technical and interaction data
Measurement Description: Events to measure user engagement with the doorhanger UI
Data Collection Category: Interaction data
Tracking Bug #: Bug 1485651
- How long will this data be collected?
- I want this data to be collected for 6 months initially (potentially renewable).
- What populations will you measure?
-
Which release channels?
- Release 66 onwards
-
Which countries?
- N/A
-
Which locales?
- all
-
Any other filters?
- No.
- If this data collection is default on, what is the opt-out mechanism for users?
- User can opt out under about:preferences#privacy.
- Please provide a general description of how you will analyze this data.
- Follow our general practice to analyze event telemetry.
- Where do you intend to share the results of your analysis?
- To Firefox team. Results will likely be shared on a mailing list and/or Bugzilla (bug 1485651)
Assignee | ||
Comment 2•4 years ago
|
||
Chris, this is a follow-up to bug 1485650, where you did the data-review for the initial rollout of Firefox Monitor.
6 months have passed since then and it's time to renew the data-review approval. The add-on has now landed in mozilla-central (in 67) with telemetry disabled for now until we get data-review+.
I pasted in the data-review doc in the comment above this one. It's the same as before except the target release has been updated to 66 onwards (we're still shipping the add-on via Balrog for 66 but it'll be in-tree for 67).
Please let me know if there's anything else you need to know for the review.
Comment 3•4 years ago
|
||
Thanks for reaching out! For future data collection reviews please attach the review request as a file and set the data-review flag to ?.
DATA COLLECTION REVIEW RESPONSE:
Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?
https://github.com/mozilla/blurts-addon/blob/master/docs/TELEMETRY.md
In future the collections may be hosted in the m-c Scalars.yaml in which case it will appear in the usual Telemetry places (Probe Dictionary, elsewhere)
Is there a control mechanism that allows the user to turn the data collection on and off?
Standard Telemetry mechanisms apply.
If the request is for permanent data collection, is there someone who will monitor the data over time?**
N/A, is not permanent.
Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Category 2, Interaction.
Is the data collection request for default-on or default-off?
default-on.
Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?
No, only counts.
Is the data collection covered by the existing Firefox privacy notice?
Yes.
Does there need to be a check-in in the future to determine whether to renew the data?
Yes. :nhnt11, you will be responsible to determine whether to renew or remove these probes before they expire.
Result: datareview+
Assignee | ||
Comment 4•4 years ago
|
||
Thank you! I'm going to use this bug to land a patch to enable Telemetry.
Assignee | ||
Comment 5•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Pushed by nhnt11@gmail.com: https://hg.mozilla.org/integration/autoland/rev/30eecb30d2a4 Enable Firefox Monitor telemetry. r=johannh
Comment 7•4 years ago
|
||
bugherder |
Comment 8•4 years ago
|
||
Is this something you wish to turn on in 67? Please request uplift to beta if yes.
Assignee | ||
Comment 9•4 years ago
|
||
Yes, it is. I'll be requesting uplift next week, along with bug 1531838. It's unfortunate that these didn't make it into 67 without an uplift, but the test failure in the other bug should be solved now. Thanks for the heads up.
Comment 10•4 years ago
|
||
Tracking both this bug and bug 1531838 to keep it on my radar as there are potential uplifts.
Assignee | ||
Comment 11•4 years ago
|
||
Comment on attachment 9051984 [details]
Bug 1525977 - Enable Firefox Monitor telemetry. r=johannh
Beta/Release Uplift Approval Request
- Feature/Bug causing the regression: Bug 1525519
- User impact if declined: No telemetry data on user interaction with Firefox Monitor UI - impacts our ability to determine the effectiveness of the feature.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It just enables telemetry recording on release.
- String changes made/needed: none
Comment 12•4 years ago
|
||
Comment on attachment 9051984 [details]
Bug 1525977 - Enable Firefox Monitor telemetry. r=johannh
Uplift approved for 67 beta 12, thanks.
Comment 13•4 years ago
|
||
bugherderuplift |
Updated•4 years ago
|
Description
•