grafana / loki

Like Prometheus, but for logs.
https://grafana.com/loki
GNU Affero General Public License v3.0
22.71k stars 3.3k forks source link

fix: support multi-zone ingesters when converting global to local limits for streams in limiter.go #13321

Closed JordanRushing closed 4 days ago

JordanRushing commented 6 days ago

What this PR does / why we need it:

Updates the calculation for stream limits to support multi-zone ingester deployments while maintaining the old logic for single-zone deployments.

re:

Which issue(s) this PR fixes:

N/A

Special notes for your reviewer:

Requires https://github.com/grafana/dskit/pull/526 to be merged and the version of dskit used by Loki to be updated to support HealthyInstancesInZoneCount()

Checklist