Skip to content

Conversation

@cathteng
Copy link
Member

When an org is deleted before the sentry app install deletion task, the deletion gets stuck. This resolves SENTRY-5ARE and will be completely fixed when I replace the RPC call in the deletion task with an outbox on sentry app install delete

@cathteng cathteng requested review from a team as code owners October 21, 2025 23:02
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 21, 2025
@cathteng cathteng enabled auto-merge (squash) October 21, 2025 23:14
@cathteng cathteng merged commit 0f5ff01 into master Oct 21, 2025
69 checks passed
@cathteng cathteng deleted the cathy/eco/unstuck-sentry-app-install-deletion branch October 21, 2025 23:25
@codecov
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ntry/deletions/defaults/sentry_app_installation.py 60.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #101895      +/-   ##
===========================================
+ Coverage   79.37%    80.95%   +1.58%     
===========================================
  Files        8724      8726       +2     
  Lines      388098    388141      +43     
  Branches    24607     24607              
===========================================
+ Hits       308062    314231    +6169     
+ Misses      79690     73564    -6126     
  Partials      346       346              

@github-actions github-actions bot locked and limited conversation to collaborators Nov 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants