Closed Bug 1438747 Opened 3 years ago Closed 2 years ago
Add missing UPDATE
_DOWNLOAD _CODE _UNKNOWN telemetry histogram
2.91 KB, text/plain
24.58 KB, patch
|Details | Diff | Splinter Review|
Bug 1438747 - Add missing app update UPDATE_DOWNLOAD_CODE_UNKNOWN telemetry histogram and missing bug numbers. r?bytesized,chutten
46 bytes, text/x-phabricator-request
|Details | Review|
This was missed when adding the histograms in bug 1137447.
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)
Comment on attachment 8951500 [details] [diff] [review] patch rev1 Changing review to chutten
Attachment #8951500 - Flags: feedback?(rweiss) → feedback?(chutten)
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+
I'll put this up on mozphab after the data collection review
Attachment #8951500 - Attachment is obsolete: true
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+
The addition of UPDATE_DOWNLOAD_CODE_UNKNOWN was missed in bug 1137447 Passed data collection review
Pushed by firstname.lastname@example.org: 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
You need to log in before you can comment on or make changes to this bug.