Crash reporting overhaul: Difference between revisions

Jump to navigation Jump to search
Change the Glean-based ping plan to go with a more incremental approach
(→‎Plan: made it clear that this won't be replacing legacy telemetry at first)
(Change the Glean-based ping plan to go with a more incremental approach)
Line 249: Line 249:
This migration requires several steps with changes happening in different parts
This migration requires several steps with changes happening in different parts
of the codebase:
of the codebase:
* First of all the new Glean-based crash ping needs to be designed. Some parts of it might require adding a new metric type to Glean's (e.g. stack traces)
* Prepare the design of a minimal crash ping that can be implemented using existing client- and server-side machinery.
* Once this is done the code in the CrashManager that assembles the ping for child process crashes needs to be adjusted to use Glean in addition to legacy telemetry
* Add support for this minimal Glean-based crash ping to Firefox desktop (inside the CrashManager) and Fenix where it needs to be done from scratch.
* The crash reporter client needs to be instructed to send Glean crash pings in addition to legacy telemetry pings. Currently Glean doesn't support C++ so this work will need to happen after we rewrite the crash reporter client
* Once the new ping's functionality has been validated, broaden the design to include parts that might require adding a new metric type (such as stack traces) and include all of the legacy crash ping information.
* Support for the new crash ping needs to be added to Fenix
* Modify the code previously introduced to fully populate the Glean-based ping and make its payload match the legacy one. This might need extra work on the Fenix side, especially to capture stack traces.
* Eventually the crash-ping from legacy telemetry will need to be decommissioned
* Last but not least the crash reporter client needs to be instructed to send Glean crash pings in addition to legacy telemetry pings. Currently Glean doesn't support C++ so this work will need to happen after we rewrite the crash reporter client.
* Decommission the legacy telemetry crash ping and remove the relevant code from Firefox desktop.


== minidump-analyzer ==
== minidump-analyzer ==
Confirmed users
417

edits

Navigation menu