Colleen O'Rourke

[Epic] Rich Slack Notifications

2

[Product Spec](https://www.notion.so/sentry/Rich-Slack-Notifications-f41a8d39e97f4cc68d0812a1413c5882) [Tech Spec](https://www.notion.so/sentry/Rich-Slack-Notifications-Tech-Spec-867736397b4f4ef29b1d0251f5ae98c8#38995e2f5d1a4ba4adf598e6842d6e6d) ### Convert notifications to use block kit ```[tasklist] - [ ] https://github.com/getsentry/sentry/issues/61456 - [ ] https://github.com/getsentry/sentry/issues/61457 - [ ] https://github.com/getsentry/sentry/issues/61459 - [ ] https://github.com/getsentry/sentry/issues/61460 - [...

ref(rules): Update apply_rule to enqueue rules with slow conditions

1

Update `apply_rule` to add rules with "slow" conditions (aka conditions that query Snuba) to a buffer to be evaluated in a task. If a rule with a slow condition has...

Follow up to https://github.com/getsentry/sentry/pull/69830#discussion_r1586723946 to check the `RuleSnooze` table before firing a delayed rule, on the off chance it got muted in the < 1 minute it took to process....

ref(rules): Remove inner lock and add logging on exception

1

Part of debugging the delayed rule processor lead here, we don't need the inner lock (it's locked [here](https://github.com/getsentry/sentry/blob/370e78ca10dddc2b018d1e9b6e41e501386cc0d2/src/sentry/buffer/redis.py#L316-L318) before `process_batch` is called).

ref(rules): Fix task name for delayed rule processing

1

The task for processing delayed rules wasn't imported into `CELERY_IMPORTS` and the name was incorrect - this fixes both and adds a log message if the buffer hook registry isn't...

ref(rules): Handle improperly formatted rule

3

One customer has a couple improperly formatted alert rules that are causing this error: https://sentry.sentry.io/issues/5245475486/. I've already edited the serializer so it's not possible to get into this state again...

chore(rules): Add metrics on Redis operations

1

Add Datadog metrics when we perform a Redis operation.

Undoing https://github.com/getsentry/sentry/pull/70986 since we're seeing data in the buffer now - that PR was kind of a desperate debugging attempt.

ref(tests): Add event_type to store_event

1

Unless you add stacktrace information to `data`, events are created as the "default" type rather than error. This trips me up all the time when I expect them to be...

ref(alerts): Remove INC-666 feature flag

1

The new processor is out! No more flag! Options automator PR: https://github.com/getsentry/sentry-options-automator/pull/2084