Cache LabelValues about thanos HOT 8 CLOSED

Is that actually true? As far as I can tell from code, this only hits all blocks' index files which are already preloaded on store node, right?

If that's so, we can close this issue. (:

from thanos.

Yes, but we are still considering not caching those full files right? My thinking was even with, we could be talking about tens of thousands of blocks at some point. And that might still get expensive then.
We also always have to do a full fanout to every single Prometheus server.

But you are generally right, this issue was probably opened as a pre-mature concern.

from thanos.

Ah, sorry, you're right. We will soon move out from this preload-all-index-files heaven.

from thanos.

Mh, I'll close this for now. On store nodes that data is served entirely from memory now anyway. The full fanout is probably not an issue anymore with that.

from thanos.

We are exploring loading blocks on demand, so it's time to reopen this as values would be no longer in memory. (:

Worth to note that mostly __name__ arg is heavily used, so we might want to cache all of it. Not sure if worth on Querier side.

The alternative would be implementing the time limit on this. Also related to the: prometheus/prometheus#6178

from thanos.

@bwplotka

The alternative would be implementing the time limit on this. Also related to the: #39

couldn't help noticing this issue is related to itself, surely a mistake :)

from thanos.

yup, fixed!

from thanos.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from thanos.