Closed
Bug 1295555
Opened 8 years ago
Closed 8 years ago
Lower client-side ping archive retention period
Categories
(Toolkit :: Telemetry, defect, P1)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox52 | --- | fixed |
People
(Reporter: gfritzsche, Assigned: gfritzsche)
Details
(Whiteboard: [measurement:client])
Attachments
(1 file)
5.58 KB,
patch
|
Dexter
:
review+
|
Details | Diff | Splinter Review |
Currently we archive all Telemetry pings for 180 days on the client. This has non-zero impact (startup disk I/O, disk fragmentation, ...). After having this out for a while, we should consider if we still need this or whether we should restrict the retention period (for all or specific ping types).
Assignee | ||
Comment 1•8 years ago
|
||
The archive can serve the following purposes: (1) power about:healthreport, looking 35 days back [0] (2) dev & qa mechanism, look up past ping data via about:telemetry etc. - looking back 7-14 days should be sufficient (3) transparency, offering users insight into past data sent - how far back is this useful? (4) (planned?) power shield/self-support - are recipes looking at archived ping data still planned? how many days back would they look at archived data? and which ping types? @bsmedberg: Do you have an opinion on (3)? @glind: are there plans that would inform (4)? 0: https://github.com/mozilla/fhr-jelly/blob/master/js/data_v4.js#L298
Flags: needinfo?(glind)
Flags: needinfo?(benjamin)
Comment 2•8 years ago
|
||
What is the actual impact? Having more history will be helpful for bug reporting/SHIELD, and matches our server behavior, but if there is obvious impact we can consider lowering it.
Flags: needinfo?(benjamin)
Assignee | ||
Comment 3•8 years ago
|
||
The direct impact i know about is: * side effects like bug 1250357 & FS fragmentation * (negligible) increased disk use There will also be background disk activity from scanning/stat()ing the archive files, which i assume we'd like to avoid for e.g. power/battery reasons. If we are actually planning to use the ping archive for 180 days, then it seems fine. If we are not planning to use the full archive, it's trivial to lower this (and likewise to increase it again in the future).
>> (4) (planned?) power shield/self-support - are recipes looking at archived ping data still planned? how many days back would they look at archived data? and which ping types?
It is not well enough planned to answer yet. If you have clear reasons to reduce it now, I will not oppose. I can't make a convincing case for keeping them on the client for 180 days.
Things we know for sure:
- some things will react to current environment (addons installed, prefs set)
- some things should be able to react to recent changes (for repair). Claim: recent is << 35 days, so that is fine.
- I have no specific ideas about how to use 'old history'. I wonder if 'under the last version' is a useful time period or not.
Summary stats that would be nice to store / access on the client:
- crashes recently (that might be 6 months worth!)
- some summaries of site diversity / page views
Flags: needinfo?(glind)
Assignee | ||
Comment 5•8 years ago
|
||
From the above, i'm planning to drop the archive retention period to 60 days. If we need more for specific use cases or pings, i'd be happy to raise this again.
Assignee | ||
Updated•8 years ago
|
Priority: P3 → P2
Assignee | ||
Updated•8 years ago
|
Priority: P2 → P1
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8798032 -
Flags: review?(alessio.placitelli)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → gfritzsche
Status: NEW → ASSIGNED
Updated•8 years ago
|
Attachment #8798032 -
Flags: review?(alessio.placitelli) → review+
Assignee | ||
Comment 7•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=644462ed353ec53cf03904819f6851f086d77466
Assignee | ||
Updated•8 years ago
|
Points: --- → 1
Pushed by georg.fritzsche@googlemail.com: https://hg.mozilla.org/integration/fx-team/rev/089590fd2754 Lower client-side ping archive retention period to 60 days. r=dexter
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/089590fd2754
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in
before you can comment on or make changes to this bug.
Description
•