Remove the DEVTOOLS_OS_* probes from our Telemetry

RESOLVED FIXED in Firefox 56

Status

()

Firefox
Developer Tools
P3
normal
RESOLVED FIXED
11 months ago
2 days ago

People

(Reporter: clarkbw, Assigned: Vangelis Katsikaros (unavailable until May '18))

Tracking

({good-first-bug})

unspecified
Firefox 56
good-first-bug
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

11 months ago
The data retrieved from these probes isn't being used and is likely more problematic than the system info we get from the telemetry ping.  

OS_ENUMERATED
OS_IS_64_BITS

This isn't urgent, but when someone gets around to it we don't need this anymore.  I'm not using it in my analysis.  Thanks!
(Reporter)

Comment 1

11 months ago
Along with this we might as well get rid of the DEVTOOLS_TABS_OPEN_* and DEVTOOLS_TABS_PINNED_* probes as well.  While interesting I don't think its answering useful questions for us anymore.

Comment 2

11 months ago
DevTools bug triage (filter on CLIMBING SHOES).
Keywords: good-first-bug
Priority: -- → P3
Assignee: nobody → vkatsikaros
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)

Updated

11 months ago
Attachment #8872697 - Flags: review?(pbrosset) → review?(mratcliffe)
Comment on attachment 8872697 [details]
Bug 1366102 - Remove the DEVTOOLS_OS_* probes from our Telemetry.

https://reviewboard.mozilla.org/r/144236/#review148352

Very clean... thanks for the hard work!
Attachment #8872697 - Flags: review?(mratcliffe) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Hi Michael, I pushed a rebased version, but I didn't spot any differences (checked again to see if the patch still makes sense). I don't think I can land the commit, so if the work is still needed you probably have to trigger this yourself :)
Flags: needinfo?(mratcliffe)
I have triggered a new try run with your rebased patch... if the patch passes we should land it.

Comment 9

9 months ago
Pushed by mratcliffe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cbb8ee4069d5
Remove the DEVTOOLS_OS_* probes from our Telemetry. r=miker

Comment 10

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/cbb8ee4069d5
Status: ASSIGNED → RESOLVED
Last Resolved: 9 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Flags: needinfo?(mratcliffe)
You need to log in before you can comment on or make changes to this bug.