Validate incoming health ping data (nightly)

RESOLVED FIXED

Status

()

P1
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: katejimmelon, Assigned: katejimmelon)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

a year ago
This include:
1. Validate input data.
2. Compute Health pings per client.
3. Compute Health pings per day. 
4. Compute Health pings per client per day. 
5. Show sendFailure and pingDiscardedForSize distribution.
(Assignee)

Updated

a year ago
Assignee: nobody → kustiuzhanina
(Assignee)

Comment 1

a year ago
Georg, could you, please, make a review?
PR https://github.com/mozilla/mozilla-reports/pull/72
Flags: needinfo?(gfritzsche)
Flags: needinfo?(gfritzsche)
Summary: Analyze Health ping data → Validate incoming health ping data
(In reply to Kate Ustiuzhanina from comment #1)
> Georg, could you, please, make a review?
> PR https://github.com/mozilla/mozilla-reports/pull/72

Chris, a heads-up on this report. This is relevant to what you are looking into as well.
Flags: needinfo?(chutten)

Comment 3

a year ago
I look forward to seeing an updated report with your changes included.

I would appreciate a note about possible reasons the data from the health ping might differ from data included in the main ping. For instance, the difference in proportions of failure types (https://mzl.la/2uUVUwy) which might just be a matter of an even more pronounced time-of-arrival effect as per bug 1371312 comment 0
Flags: needinfo?(chutten)
(Assignee)

Updated

a year ago
Summary: Validate incoming health ping data → Validate incoming health ping data (nightly)
(Assignee)

Comment 5

a year ago
Hi Chris, I updated PR (https://github.com/mozilla/mozilla-reports/pull/72), could you, please, take a look?
Flags: needinfo?(chutten)
(Assignee)

Comment 6

a year ago
(In reply to Kate Ustiuzhanina from comment #5)
> Hi Chris, I updated PR (https://github.com/mozilla/mozilla-reports/pull/72),
> could you, please, take a look?

I updated code according your comments (or made some answers).

Updated

a year ago
Flags: needinfo?(chutten) → needinfo?(kustiuzhanina)
(Assignee)

Updated

a year ago
Blocks: 1393731
(Assignee)

Updated

a year ago
Flags: needinfo?(kustiuzhanina) → needinfo?(chutten)

Updated

a year ago
Flags: needinfo?(chutten) → needinfo?(kustiuzhanina)
(Assignee)

Comment 7

a year ago
Fixed in PR https://github.com/mozilla/mozilla-reports/pull/72
Status: NEW → RESOLVED
Last Resolved: a year ago
Flags: needinfo?(kustiuzhanina)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.