Repository navigation
feat(sidekiq): add opt-in ActiveJob reporting through sentry-rails - #3134
Open
solnic wants to merge 1 commit into
Conversation
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 13, 2026 13:23
4586b71 to
412cdfc
Compare
solnic
marked this pull request as draft
October 13, 2026 13:25
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
2 times, most recently
from
October 14, 2026 06:13
906473e to
4f51c1c
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 14, 2026 06:31
4f51c1c to
185934b
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
3 times, most recently
from
October 14, 2026 08:23
7671f66 to
87c690c
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
2 times, most recently
from
October 14, 2026 10:34
91bbe8e to
5520b91
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 14, 2026 10:56
5520b91 to
e76091f
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 14, 2026 11:42
e76091f to
6014ce1
Compare
solnic
changed the base branch from
chore/3121-queues-module-conformance-improvements
to
chore/3119-switch-activejob-integration-to-the-backend-extension-registry
October 14, 2026 11:44
solnic
marked this pull request as ready for review
October 14, 2026 11:49
solnic
marked this pull request as draft
October 15, 2026 08:25
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 15, 2026 12:45
84ac741 to
97641bf
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 15, 2026 12:51
97641bf to
9a3e4f4
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 15, 2026 13:37
9a3e4f4 to
2768792
Compare
solnic
marked this pull request as draft
October 20, 2026 08:52
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 20, 2026 12:44
2d1682c to
29d356b
Compare
solnic
marked this pull request as ready for review
October 20, 2026 12:48
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 20, 2026 12:57
29d356b to
f14ea37
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
3 times, most recently
from
October 23, 2026 10:52
2d0869f to
bb45494
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
October 23, 2026 11:49
bb45494 to
04e3b14
Compare
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
November 2, 2026 15:56
04e3b14 to
2f85024
Compare
solnic
marked this pull request as draft
November 2, 2026 15:58
solnic
force-pushed
the
chore/2937-migrate-sentry-sidekiq-to-aj-first-integration
branch
from
November 2, 2026 15:59
2f85024 to
64c07a7
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 336b959. Configure here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

sentry-sidekiq can now leave ActiveJob jobs running on Sidekiq to sentry-rails. By default
ActiveJob::QueueAdapters::SidekiqAdapterstays inconfig.rails.skippable_job_adapters, so sentry-sidekiq keeps reporting them. Rails apps opt in by removing it:Once opted in,
Sentry::Sidekiq::ActiveJobExtensionreports those jobs with aSidekiq/<JobClass>transaction name, theauto.queue.sidekiqorigin andSentry::Sidekiqas the integration. The client middleware still records thequeue.publishspan. sentry-sidekiq still reports errors raised before the job starts.report_after_job_retries,report_only_dead_jobsandattempt_thresholdonly apply to those errors.Job extensions
A Sidekiq process runs plain Sidekiq workers and ActiveJob jobs wrapped in ActiveJob's
JobWrapper. A job is reported either by sentry-sidekiq's middleware, which sees the Sidekiq job hash, or by sentry-rails' ActiveJob hook, which sees theActiveJob::Baseinstance. Each extension covers one of those cases:Sentry::Sidekiq::JobExtensionSentry::Sidekiq::ActiveJobSupport::JobExtension"wrapped"Sentry::Sidekiq::ActiveJobExtensionActiveJob::BaseinstanceActiveJobSupport::JobExtensionsubclassesJobExtensionand only adds the ActiveJob handling these jobs already get today: naming them after the wrapped class and stripping ActiveJob's_aj_keys from the error context.ActiveJobExtensionsubclassesSentry::Rails::ActiveJobExtensionand keeps Sidekiq's identity. The two ActiveJob extensions receive different objects and need different superclasses, so they stay separate.Closes #3122
Closes #2937