Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(feedback): add feedback-events DLQ #238

Merged
merged 5 commits into from Mar 15, 2024
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions CODEOWNERS
Validating CODEOWNERS rules …
Expand Up @@ -9,6 +9,7 @@
/topics/ingest-events-dlq.yaml @getsentry/owners-snuba
/topics/ingest-metrics-dlq.yaml @getsentry/owners-snuba
/topics/ingest-generic-metrics-dlq.yaml @getsentry/owners-snuba
/topics/ingest-feedback-events-dlq.yaml @getsentry/owners-snuba
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this replays team now?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh I did this because of the pattern that all other dlq's are owners-snuba only. Would you recommend I change it to snuba+replay, or just replay?

Copy link
Member

@lynnagara lynnagara Mar 15, 2024

Choose a reason for hiding this comment

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

Hmm, probably just replay is fine if you guys own it. Or whatever matches the main ingest-feedback-events topic I guess.


# Topics consumed by Snuba
/topics/events.yaml @getsentry/owners-snuba
Expand Down
17 changes: 17 additions & 0 deletions topics/ingest-feedback-events-dlq.yaml
@@ -0,0 +1,17 @@
topic: ingest-feedback-events-dlq
pipeline: user-feedback
description: DLQ for ingest-feedback-events
services:
producers:
Copy link
Member

Choose a reason for hiding this comment

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

what does producers signify?

Copy link
Member

Choose a reason for hiding this comment

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

Just which services (i.e. sentry) are producing to this topic. Likely will be used soon to make these topics automatically available to sentry code via configuration.

- getsentry/sentry
consumers: []
schemas:
- version: 1
compatibility_mode: none
type: json
resource: any.json
examples:
- any/
topic_creation_config:
compression.type: lz4
log.retention.minutes: "10080" # 7 days