Skip to content

Diversify errors in Error Tracking

Peter Leitzen requested to merge pl-error-caller into main

What does this merge request do and why?

Error Tracking groups errors by the following criteria:

  • Exception class
  • Location of first backtrace frame

Since Telemetry.capture_exception is called by GDK::Output.error and GDK::Output.abort the errors were grouped incorrectly due to the first frame being almost always the same.

This commit drops the first frame to ensure that errors are created from distinct locations.

How to set up and validate locally

  1. Run the following GDK commands
gdk config get foo
gdk rails
  1. Watch Events increase in

Impacted categories

The following categories relate to this merge request:

Merge request checklist

  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Tests added for new functionality. If not, please raise an issue to follow-up.
  • Documentation added/updated, if needed.
  • Announcement added, if change is notable.
  • gdk doctor test added, if needed.
  • Add the ~highlight label if this MR should be included in the CHANGELOG.md.
Edited by Peter Leitzen

Merge request reports

Loading