Closed
Bug 1290155
Opened 8 years ago
Closed 8 years ago
List most recent weekly report as option in update orphaning dashboard
Categories
(Toolkit :: Application Update, defect, P4)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
People
(Reporter: spohl, Assigned: mhowell)
References
Details
Once bug 1290146 is fixed we can list a weekly report that is one week more recent than what we currently display in the update orphaning dashboard.
Specifically, we should remove three lines in the update orphaning's index.html, starting at [1].
[1] https://github.com/mozilla/telemetry-dashboard/blob/221d4b42f6e79dac7cc134e0011a15d6fbaa239d/update-orphaning/index.html#L120
Updated•8 years ago
|
Priority: -- → P4
Reporter | ||
Comment 1•8 years ago
|
||
This has become actionable now that bug 1290146 is fixed and will allow us to display the most recent week as an option in the dashboard (instead of being two weeks behind).
Setting n-i just to get eyes on this.
Flags: needinfo?(robert.strong.bugs)
Comment 2•8 years ago
|
||
I saw that and already manually ran the queries to verify.
I noticed that the dashboard uses more than a week of data. Is that intentional? I have a few more similar questions and I'll ping you on irc about them.
Flags: needinfo?(robert.strong.bugs) → needinfo?(spohl.mozilla.bugs)
Reporter | ||
Comment 3•8 years ago
|
||
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #2)
> I saw that and already manually ran the queries to verify.
>
> I noticed that the dashboard uses more than a week of data. Is that
> intentional? I have a few more similar questions and I'll ping you on irc
> about them.
Yes, this is intentional. We initially meant to restrict this to a range of 7 days, starting 17 days before the creation date of the longitudinal dataset. However, the goal was simply to give clients enough time to send telemetry data so we removed the restriction of the maximum range. Each weekly report is now looking at clients who reported data in the past 17 days.
The notebook may still refer to a 7 day range in one of the comments. That could and should be changed to avoid confusion.
Flags: needinfo?(spohl.mozilla.bugs)
Comment 4•8 years ago
|
||
Since it is a sliding period and those clients will report that data at some point anyways what value does this provide over a strict date period? Is it just more data?
Reporter | ||
Comment 5•8 years ago
|
||
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #4)
> Is it just more data?
This.
Assignee | ||
Comment 6•8 years ago
|
||
Assignee: nobody → mhowell
Assignee | ||
Comment 7•8 years ago
|
||
Above PR has been merged.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•