grafana / loki

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

fix(blooms): Skip multi-tenant TSDBs during bloom planning #14770

Closed salvacorts closed 2 weeks ago

salvacorts commented 2 weeks ago

What this PR does / why we need it: Tables that have not been fully compacted yet will have multi-tenant TSDBs for which the tenant is "" (coming tsdbStore.UsersForPeriod and indexStorageClient.ListFiles)

In this PR we update the planner to skip these multi-tenant TSDBs.

I though about fixing this at the indexStorageClient.ListFiles level but I think it's relatively risky given it's is used in the compactor.

Checklist