Closed
Bug 1438747
Opened 7 years ago
Closed 6 years ago
Add missing UPDATE_DOWNLOAD_CODE_UNKNOWN telemetry histogram
Categories
(Toolkit :: Application Update, enhancement, P2)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
Attachments
(3 files, 1 obsolete file)
This was missed when adding the histograms in bug 1137447.
Assignee | ||
Comment 1•7 years ago
|
||
Hi Rebecca, this adds the missing UPDATE_DOWNLOAD_CODE_UNKNOWN histogram that covers the case when the patch type is unknown for the download code and should have been added in bug 1137447. I also added bug numbers for all of the app update histograms. Thanks!
Attachment #8951500 -
Flags: feedback?(rweiss)
Assignee | ||
Updated•7 years ago
|
Priority: -- → P2
Assignee | ||
Comment 2•7 years ago
|
||
Comment on attachment 8951500 [details] [diff] [review]
patch rev1
Changing review to chutten
Attachment #8951500 -
Flags: feedback?(rweiss) → feedback?(chutten)
Comment 3•7 years ago
|
||
Comment on attachment 8951500 [details] [diff] [review]
patch rev1
Review of attachment 8951500 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks for adding bug_numbers in.
I don't suppose these underwent Data Collection Review in bug 1137447? I looked around but didn't see it.
I'd grandfather them in, but they're non-expiring -and- opt-out. Could you run through the process at https://wiki.mozilla.org/Firefox/Data_Collection ? I can be the Data Steward and get you a (supposing I can stop needing days off for annoying head colds) quick turn-around on the review.
Attachment #8951500 -
Flags: feedback?(chutten) → feedback+
Assignee | ||
Comment 4•6 years ago
|
||
Attachment #9014957 -
Flags: review?(chutten)
Assignee | ||
Comment 5•6 years ago
|
||
I'll put this up on mozphab after the data collection review
Assignee | ||
Updated•6 years ago
|
Attachment #8951500 -
Attachment is obsolete: true
Comment 6•6 years ago
|
||
Comment on attachment 9014957 [details]
data collection review request
Preliminary note:
At some point in the future you may be interested to convert the enumerated histograms to categorical ones. That way the labels are part of the probe, and extending them after the fact becomes easier, like for DEVTOOLS_ENTRY_POINT: https://mzl.la/2OIYynd
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?
Yes. Standard Telemetry mechanisms apply.
Is there a control mechanism that allows the user to turn the data collection on and off?
Yes. Standard Telemetry mechanisms apply.
If the request is for permanent data collection, is there someone who will monitor the data over time?
Yes, :rstrong.
Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Category 1
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.
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?
No, this is permanent collection.
---
Result: datareview+
Attachment #9014957 -
Flags: review?(chutten) → review+
Assignee | ||
Comment 7•6 years ago
|
||
The addition of UPDATE_DOWNLOAD_CODE_UNKNOWN was missed in bug 1137447
Passed data collection review
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9ed28d8b07df
Add missing app update UPDATE_DOWNLOAD_CODE_UNKNOWN telemetry histogram and missing bug numbers. r=chutten,bytesized
Comment 9•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•