Skip to content

Conversation

@cmanallen
Copy link
Member

@cmanallen cmanallen commented Aug 27, 2025

Checks the global config to see if it should or should not publish the replay-event to Snuba. For now the option should always be false until the recording consumer can be updated to handle publishing to Snuba.

Related: getsentry/sentry-kafka-schemas#434
Related: getsentry/sentry#98353

@cmanallen cmanallen changed the title ref(replays): Add option gating snuba publishing ref(replays): Add option gating Snuba publishing to ingest-replay-events Aug 27, 2025
Copy link
Member

@Dav1dde Dav1dde left a comment

Choose a reason for hiding this comment

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

Maybe you can find an existing Replay integration test you can modify to show that this works.

@cmanallen cmanallen added this pull request to the merge queue Aug 28, 2025
Merged via the queue into master with commit f2688fc Aug 28, 2025
47 of 48 checks passed
@cmanallen cmanallen deleted the cmanallen/replays-global-config-snuba-gate branch August 28, 2025 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants