Skip to content

Commit

Permalink
Improve SubprocessFailed Sentry errors
Browse files Browse the repository at this point in the history
* Add a more meaningful title so they are distinguishable in the main
  index.

* Copy the original backtrace.
  • Loading branch information
deivid-rodriguez committed Dec 7, 2022
1 parent 3005635 commit 32846c4
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions updater/lib/dependabot/updater.rb
Original file line number Diff line number Diff line change
Expand Up @@ -854,11 +854,10 @@ def handle_dependabot_error(error:, dependency:)
# info such as file contents or paths. This information is already
# in the job logs, so we send a breadcrumb to Sentry to retrieve those
# instead.
msg = "Dependency update process failed, please check the job logs"
Raven.capture_exception(
SubprocessFailed.new(msg, raven_context: error.raven_context),
raven_context
)
msg = "Subprocess #{error.raven_context[:fingerprint]} failed to run. Check the job logs for error messages"
sanitized_error = SubprocessFailed.new(msg, raven_context: error.raven_context)
sanitized_error.set_backtrace(error.backtrace)
Raven.capture_exception(sanitized_error, raven_context)

{ "error-type": "unknown_error" }
when *Octokit::RATE_LIMITED_ERRORS
Expand Down

0 comments on commit 32846c4

Please sign in to comment.