getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.13k stars 4.2k forks source link

Spike Protection Algorithm allowed error spike #76751

Open kpujjigit opened 2 months ago

kpujjigit commented 2 months ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

Spike in error events occurs for a single project with spike protection enabled

Expected Result

Spike protection should identify occurrences of spike based on healthy ingestion.

Actual Result

Spike protection actually increased upper bound at the sight of a spike and allowed it. the threshold then held at a very high value long after the spike subsided.

Image

Image

Product Area

Settings - Spike Protection

Link

No response

DSN

No response

Version

No response

┆Issue is synchronized with this Jira Improvement by Unito

getsantry[bot] commented 2 months ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 2 months ago

Routing to @getsentry/product-owners-settings-spike-protection for triage ⏲️

brianthi commented 2 months ago

Thanks for the feedback. We don't have plans at the moment to make changes to the Spike Detection behavior but I've passed this along to our PM and we'll reference this when considering updating the algorithm. For more information on how Spike Protection is currently computed, reference our docs here: https://docs.sentry.io/pricing/quotas/spike-protection/.