Closed
Bug 1298887
Opened 8 years ago
Closed 8 years ago
Validate that child telemetry still works after bug 1218576
Categories
(Toolkit :: Telemetry, defect, P3)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
People
(Reporter: chutten, Assigned: chutten)
References
(Blocks 1 open bug)
Details
(Whiteboard: [measurement:client])
After bug 1218576 lands, we need to make sure nothing's gone stupid.
Some examples:
1) Keep an eye out for telemetry-alerts. It'll take three days to get there, but I expect absolutely everything would explode if we got aggregator stuff wrong.
2) After one day, check telemetry.mozilla.org. Check parent-only and child-only metrics to make sure they're similar to what came before (subject to the changes as documented on the mailing list (the aggregate measures might be different for child-reported measures because the meaning of "count" changes))
3) After a couple of days, run the E10sExperimentView against nightly. Make sure the result can be run meaningfully through the e10s_experiment.ipynb
4) After one day, run a notebook and check that get_pings from the affected build dates returns full and meaningful results. Verify that get_pings crossing the gap (before and after the build with the change) returns meaningful results as well.
Comment 1•8 years ago
|
||
I think we should also check the main ping payloads after the landings verifying that:
* histograms & keyedHistograms moved
* the new processes/content section has the expected keys & correctly typed content etc.
Updated•8 years ago
|
Priority: -- → P3
Whiteboard: [measurement:client]
Assignee | ||
Comment 2•8 years ago
|
||
> 4) After one day, run a notebook and check that get_pings from the affected build dates returns full and meaningful results. Verify that get_pings crossing the gap (before and after the build with the change) returns meaningful results as well.
get_pings reports pings from builds before and after the build with correct numbers (I checked my own clientId). This criteria's a pass.
>* histograms & keyedHistograms moved
>* the new processes/content section has the expected keys & correctly typed content etc.
Raw ping documents look good to me. I consider these passed as well.
Not going to check tmo until tomorrow morning's aggregator run.
Assignee | ||
Comment 3•8 years ago
|
||
> 2) After one day, check telemetry.mozilla.org. Check parent-only and child-only metrics to make sure they're similar to what came before (subject to the changes as documented on the mailing list (the aggregate measures might be different for child-reported measures because the meaning of "count" changes))
Looking pretty solid, actually: https://mzl.la/2dnq1FH
Assignee | ||
Comment 4•8 years ago
|
||
Everything checks out!
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•