[EPIC] Phase out low priority symbolication

getsentry / sentry

Developer-first error tracking and performance monitoring

https://sentry.io

Other

38.71k stars 4.15k forks source link

[EPIC] Phase out low priority symbolication #76786

Open loewenheim opened 1 month ago

loewenheim commented 1 month ago

Since Symbolicator autoscales now, we hopefully shouldn't need the LPQ machinery anymore.

Things that remain to be done:

[ ] Set SENTRY_REALTIME_METRICS_BACKEND to "sentry.processing.realtime_metrics.dummy.DummyRealtimeMetricsStore" in getsentry, as it is in sentry.
[ ] Remove should_demote_symbolication.
[ ] Remove record_symbolication_duration.
[ ] Remove the realtime metrics code.
[ ] Remove LPQ from symbolication logic.
[ ] Remove queues, workers, deployments.

loewenheim commented 1 month ago

The first step is https://github.com/getsentry/sentry/pull/76788.

loewenheim commented 4 weeks ago

Increase JS Symbolicator max replicas to absorb LPQ traffic https://github.com/getsentry/ops/pull/11989

loewenheim commented 3 weeks ago

We're currently running on 80% of LPQ traffic going to normal Symbolicators. Before we switch it off entirely, we should increase Symbolicator autoscaling https://github.com/getsentry/ops/pull/12039.