Closed Bug 1555352 Opened 5 years ago Closed 5 years ago

Validate data after clearing metrics on disabling lands

Categories

(Data Platform and Tools :: Glean: SDK, task, P1)

task

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mdroettboom, Assigned: mdroettboom)

References

Details

(Whiteboard: [telemetry:mobilesdk:m8])

We should validate the incoming data looking for any weird behaviors due to landing the feature that clears metrics when they are disabled.

Things to check include:

  • Pings with the c0ffee client_id.

  • Pings sent without required application lifetime metrics (which will probably be rejected by the JSON schema)

:chutten has more background in this and could probably suggest more good things to check.

Type: defect → task
Flags: needinfo?(chutten)
Assignee: nobody → mdroettboom
Whiteboard: [telemetry:mobilesdk:m?] → [telemetry:mobilesdk:m8]

I'm afraid I'm missing context on what it means that metrics are cleared when reporting is disabled. Which metrics? Cleared how?

Flags: needinfo?(chutten)

When the user disables (opts-out) of telemetry, most metrics persisted to disk and in memory at that point are cleared, and the client_id is set to the infamous "c0ffee" value. More details here.

chutten: I have needinfo'd you thinking you might have experience verifying such things on desktop so could fill in some more things we should look at. If you have nothing to add, no worries :)

Flags: needinfo?(chutten)

For just the "clearing things out" part, I think you have it covered: Check for the canary, Check that things sent after opting back in have the newly-created data...

Might be worth double-checking that it doesn't get tripped too frequently. If you see the client count inflate when this lands that could be evidence that the client id is being reset too often, for example. This would also exhibit as a left-shift in the distribution of seq numbers and sequence lengths overall.

Flags: needinfo?(chutten)
Priority: -- → P1
Blocks: 1552884
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.