Closed Bug 1966213 Opened 1 year ago Closed 1 year ago

Fall back to Glean client_info for crash.app_channel and crash.app_display_version

Categories

(Crash Pings :: Post-Processing, enhancement)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: afranchuk, Assigned: afranchuk)

References

Details

Attachments

(1 file)

We use crash.app_channel and crash.app_display_version to select pings since https://github.com/mozilla/crash-ping-ingest/commit/3509549f86bcf80258352a6adc59ba6872c43fcd (as part of some rewriting of the SQL for bug 1959459), however per bug 1966210 the Java exceptions don't populate these fields. Just in case these metrics (or crash annotations which populate them) are missing, use the client_info for those values, as that should be close enough and it is more detrimental to exclude pings than it is to potentially attribute them to the wrong version (the volume of misattribution is likely to be very, very low, if existing at all).

The backfill of crash ping data (with fixed ingestion) is complete, and frontend caches are clearing now. There was some spam stuff from crawlers or otherwise in the caches, so it's taking a while to clear (I've since added some validation to prevent it, though ultimately it was just wasting tiny bits of space with empty responses).

Pushed by afranchuk@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/291866a2dde1 Document best practices and caveats for Glean client_info usage. r=gsvelto
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: