Skip to content

Conversation

@mifu67
Copy link
Contributor

@mifu67 mifu67 commented Jun 30, 2025

Since we had to roll back dual processing for dynamic alerts, re-enable processing through subscription processor.

@mifu67 mifu67 requested a review from ceorourke June 30, 2025 20:41
@mifu67 mifu67 requested a review from a team as a code owner June 30, 2025 20:41
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 30, 2025
@ceorourke
Copy link
Member

We can't turn this back on until we fix https://sentry.sentry.io/issues/6713407596/

Copy link
Member

@ceorourke ceorourke left a comment

Choose a reason for hiding this comment

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

ok ignore my other comment

@codecov
Copy link

codecov bot commented Jun 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #94652      +/-   ##
==========================================
- Coverage   87.93%   87.93%   -0.01%     
==========================================
  Files       10419    10419              
  Lines      602918   602918              
  Branches    23448    23448              
==========================================
- Hits       530148   530146       -2     
- Misses      72264    72266       +2     
  Partials      506      506              

@mifu67 mifu67 merged commit be4a527 into master Jun 30, 2025
65 checks passed
@mifu67 mifu67 deleted the mifu67/anomaly-detection/quick-fix-ACI branch June 30, 2025 21:08
@mifu67 mifu67 added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Jul 1, 2025
@getsentry-bot
Copy link
Contributor

PR reverted: 3824d9a

getsentry-bot added a commit that referenced this pull request Jul 1, 2025
mifu67 added a commit that referenced this pull request Jul 1, 2025
…4737)

Reintroducing the quick fix from
#94652 (which we reverted for
testing purposes).
andrewshie-sentry pushed a commit that referenced this pull request Jul 14, 2025
…4652)

Since we had to roll back dual processing for dynamic alerts, re-enable
processing through subscription processor.
andrewshie-sentry pushed a commit that referenced this pull request Jul 14, 2025
andrewshie-sentry pushed a commit that referenced this pull request Jul 14, 2025
…4737)

Reintroducing the quick fix from
#94652 (which we reverted for
testing purposes).
@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 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 Trigger: Revert Add to a merged PR to revert it (skips CI)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants