Skip to content

Remove flag for metric alert rate limiting #58648

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.

Sign up for GitHub

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

Jump to bottom
Closed
isabellaenriquez opened this issue Oct 23, 2023 · 0 comments · Fixed by #60487
Closed

Remove flag for metric alert rate limiting #58648

isabellaenriquez opened this issue Oct 23, 2023 · 0 comments · Fixed by #60487
Assignees
@isabellaenriquez
Milestone
Incident Remediation

Comments

@isabellaenriquez
Copy link
Member

isabellaenriquez commented Oct 23, 2023 •
edited
Loading

Rate limiting was added for metric alerts in #58032 under the options-backed feature flag "metric_alerts.rate_limit". Now we need to test this rate limiting feature internally to make sure it's working as expected and would not cause the metric alert consumer to backlog.

Once that's all done, the flag should be removed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@isabellaenriquez isabellaenriquez

Labels
None yet
Projects
None yet
Milestone
Incident Remediation
Development

Successfully merging a pull request may close this issue.

chore(alerts): Remove metric alert rate limiting flag getsentry/sentry
1 participant
@isabellaenriquez