Closed
Bug 756152
Opened 13 years ago
Closed 13 years ago
disable persistent telemetry to prevent telemetry data loss
Categories
(Toolkit :: Telemetry, defect)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: froydnj, Assigned: froydnj)
Details
(Whiteboard: [qa-])
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
taras.mozilla
:
review+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
For persistent telemetry, we only send histogram data; we don't send other information that we collect during normal telemetry operation (amount of RAM, various startup times, etc.). Due to my poor assumptions and the way telemetry data is aggregated on the server side, this approach discards (on the server side) that "extra" non-histogram information.
Loss of telemetry data is bad. Given that restructuring the server side would be rather involved, switching persistent telemetry off for the short term is a better option. We have open bugs (bug 748517 and related) for restructuring persistent telemetry to make it easier to save this extra information. While those bugs can be fixed relatively quickly, shoving through the train process on such short notice doesn't seem advisable.
Assignee | ||
Comment 1•13 years ago
|
||
Setting tracking flags. I'm new to tracking flags, so hopefully I'm not screwing it up!
status-firefox13:
--- → affected
status-firefox14:
--- → affected
status-firefox15:
--- → affected
tracking-firefox13:
--- → ?
tracking-firefox14:
--- → ?
tracking-firefox15:
--- → ?
Summary: disable persistent telemetry temporarily → disable persistent telemetry to prevent telemetry data loss
Assignee | ||
Comment 2•13 years ago
|
||
Here's a patch that just disables the sending, but leaves everything else intact. Tests still pass, thanks to the observer hook.
Attachment #624821 -
Flags: review?(taras.mozilla)
Comment 3•13 years ago
|
||
Comment on attachment 624821 [details] [diff] [review]
patch
r-, please just change the check for persistence data existence
Attachment #624821 -
Flags: review?(taras.mozilla) → review-
Assignee | ||
Comment 4•13 years ago
|
||
Disabling by simply not loading data.
Attachment #624821 -
Attachment is obsolete: true
Attachment #624972 -
Flags: review?(taras.mozilla)
Updated•13 years ago
|
Updated•13 years ago
|
Attachment #624972 -
Flags: review?(taras.mozilla) → review+
Assignee | ||
Comment 5•13 years ago
|
||
Target Milestone: --- → mozilla15
Comment 6•13 years ago
|
||
(In reply to Nathan Froyd (:froydnj) from comment #5)
> https://hg.mozilla.org/integration/mozilla-inbound/rev/82154680bc6a
Please nominate for aurora/beta approval tomorrow (5/22) and plan to land tomorrow as well so that this change makes it into FF13 beta 5.
Comment 7•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•13 years ago
|
||
Comment on attachment 624972 [details] [diff] [review]
patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 707320 and followups.
User impact if declined: None, this feature has no user interaction.
Testing completed (on m-c, etc.): Tests in the test suite.
Risk to taking this patch (and alternatives if risky): Epsilon risk that something will go wrong and telemetry won't get sent in, period. Unwilling to say zero risk. :)
String or UUID changes made by this patch: None.
Attachment #624972 -
Flags: approval-mozilla-beta?
Attachment #624972 -
Flags: approval-mozilla-aurora?
Comment 9•13 years ago
|
||
Comment on attachment 624972 [details] [diff] [review]
patch
[Triage Comment]
Understood on risk. Can you make sure to watch aurora/beta channel telemetry volume it's as expected?
Approved for Aurora 14 and Beta 13. Pleas land asap to make it into beta 5.
Attachment #624972 -
Flags: approval-mozilla-beta?
Attachment #624972 -
Flags: approval-mozilla-beta+
Attachment #624972 -
Flags: approval-mozilla-aurora?
Attachment #624972 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 10•13 years ago
|
||
http://hg.mozilla.org/releases/mozilla-aurora/rev/7bb1691988a5
http://hg.mozilla.org/releases/mozilla-beta/rev/85406204d3ca
Status: RESOLVED → UNCONFIRMED
Ever confirmed: false
Resolution: FIXED → ---
Assignee | ||
Updated•13 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Comment 11•13 years ago
|
||
Is there something QA can do to verify this fix?
Assignee | ||
Comment 12•13 years ago
|
||
Nothing for QA here.
Assignee | ||
Comment 14•13 years ago
|
||
(In reply to Alex Keybl [:akeybl] from comment #9)
> Understood on risk. Can you make sure to watch aurora/beta channel telemetry
> volume it's as expected?
Aurora and Beta telemetry volume seem as expected so far, save for a brief dip (~30-40%) on the 24th and 25th, respectively. I don't claim to understand this, seeing as how telemetry volume went up when we enabled persistent telemetry (since we were sending roughly 2x the number of pings). One would therefore expect it to decrease by the same volume...I suppose it's possible people are not updating quickly?
In any event, Nightly telemetry volume has held more-or-less steady, so I think we're OK on the telemetry front.
You need to log in
before you can comment on or make changes to this bug.
Description
•