Skip to content

Ignore internal Sidekiq::JobRetry::Skip exception#1763

Merged
st0012 merged 1 commit intomasterfrom
fix-#1731-2
Mar 18, 2022
Merged

Ignore internal Sidekiq::JobRetry::Skip exception#1763
st0012 merged 1 commit intomasterfrom
fix-#1731-2

Conversation

@st0012
Copy link
Copy Markdown
Contributor

@st0012 st0012 commented Mar 17, 2022

After consulting Sidekiq's author Mike, this exception can be ignored as an adhoc solution to #1731.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 17, 2022

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.44%. Comparing base (1e71de0) to head (a803562).
⚠️ Report is 606 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1763   +/-   ##
=======================================
  Coverage   98.44%   98.44%           
=======================================
  Files         144      144           
  Lines        8436     8438    +2     
=======================================
+ Hits         8305     8307    +2     
  Misses        131      131           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@st0012 st0012 requested a review from sl0thentr0py March 17, 2022 23:58
@st0012 st0012 merged commit ff48788 into master Mar 18, 2022
@st0012 st0012 deleted the fix-#1731-2 branch March 18, 2022 11:37
@st0012 st0012 modified the milestones: 5.3.0, 5.2.1 Mar 18, 2022
danielnolan added a commit to danielnolan/honeybadger-ruby that referenced this pull request Oct 24, 2022
* Since upgrading to Honeybadger 5.0 we are getting duplicate errors
  reported for Sidekiq::JobRetry::Skip and another reported error for
  the exception that caused the Sidekiq job to be retried.
* There was a pr merged to Sentry for a similar issue and Mike Perham
  said this exception could be ignored.
  getsentry/sentry-ruby#1763
  rails/rails#43625 (comment)
joshuap pushed a commit to honeybadger-io/honeybadger-ruby that referenced this pull request Oct 25, 2022
* Since upgrading to Honeybadger 5.0 we are getting duplicate errors
  reported for Sidekiq::JobRetry::Skip and another reported error for
  the exception that caused the Sidekiq job to be retried.
* There was a pr merged to Sentry for a similar issue and Mike Perham
  said this exception could be ignored.
  getsentry/sentry-ruby#1763
  rails/rails#43625 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants