Implement Glean-based crash pings in desktop Firefox
Categories
(Toolkit :: Crash Reporting, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox111 | --- | fixed |
People
(Reporter: gsvelto, Assigned: afranchuk)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
As per title, this would be the minimally viable ping as designed in bug 1790569 and would plug into the existing infrastructure, possibly here where we send the legacy ping.
Reporter | ||
Updated•1 year ago
|
Assignee | ||
Comment 1•1 year ago
|
||
Glean-based crash pings are sent alongside legacy crash pings.
Depends on D166925
Assignee | ||
Comment 2•1 year ago
|
||
As we are planning to support Firefox on Android as well, should the glean pings/metrics be added to the gecko_{pings,metrics}
categories rather than firefox_desktop_{pings,metrics}
? Or should we keep it as firefox_desktop
until we support Android too?
Comment 3•1 year ago
|
||
Chiming in to say: It's safe to do either of the options you propose. Putting them in gecko_*
will just create some empty tables until we start sending things from Fenix. Putting them in firefox_desktop_*
and then moving them later is supported by our infra.
Reporter | ||
Comment 4•1 year ago
|
||
Fenix is the next logical step so we can start with gecko_*
. The main driver behind this is having telemetry from Fenix too. History note: we used to have fully-featured crash pings back when Firefox for Android was Fennec, however they were lost during the transition to GeckoView.
Assignee | ||
Comment 5•1 year ago
|
||
That was my thought exactly, I've already made the change locally. I've just been waiting for data review to complete before sending the new patches.
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ff31f4ed8cfb Implement Glean-based crash pings in desktop Firefox r=gsvelto
Comment 7•1 year ago
|
||
bugherder |
Assignee | ||
Updated•1 year ago
|
Description
•