Skip to content

fix(sentry apps): Self heal hooks in background w/o org id - #123920

Merged
Christinarlong merged 2 commits into
masterfrom
crl/servicehook-self-heal
Sep 9, 2026
Merged

fix(sentry apps): Self heal hooks in background w/o org id#123920
Christinarlong merged 2 commits into
masterfrom
crl/servicehook-self-heal

Conversation

@Christinarlong

@Christinarlong Christinarlong commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Looked more into the missing servicehook sentry issue since all the broken ones from before I thought were due to the lack of outbox. But it seems like we didn't populate org_id for a time (2025-08-26 and 2026-02-18) so the lookup for hooks fails. The scope is ~100 hooks which seems low for a 6 month span but 🤷

@Christinarlong
Christinarlong requested review from a team as code owners September 9, 2026 17:07
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 9, 2026
Comment thread src/sentry/sentry_apps/tasks/sentry_apps.py

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 697311a. Configure here.

Comment thread src/sentry/sentry_apps/tasks/sentry_apps.py
Comment thread src/sentry/sentry_apps/tasks/sentry_apps.py

@cvxluo cvxluo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if the volume is low, should we consider a custom job instead?

@Christinarlong

Copy link
Copy Markdown
Contributor Author

if the volume is low, should we consider a custom job instead?

custom job was also considered but imo I prefer code change 'cause lowk less to write :hide:

@Christinarlong
Christinarlong merged commit 8a7d415 into master Sep 9, 2026
75 of 78 checks passed
@Christinarlong
Christinarlong deleted the crl/servicehook-self-heal branch September 9, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants