Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to perform runtime dedup using multiple labels for --query.replica-label flag #1347

Closed
sanamana opened this issue Jul 23, 2019 · 5 comments

Comments

@sanamana
Copy link

Context/Use Case

The querier component accepts --query.replica-label flag. Right now only one label name can be specified and it will be used to perform dedup. We have distributed our data across multiple Prometheus instances and have labeled Prometheus instances with about 3 labels.

When the data is retrieved by thanos we would like to dedup on all 3 labels.

One way to do that is to collapse the 3 labels into a single label and use it for --query.replica-label flag.

Another way is to enhance thanos, to perform dedup on multiple replica labels.

Would like to get feedback on the second approach. The second approach allows us to perform rollups on our custom labels when dedup=false is set in the query.

@brancz
Copy link
Member

brancz commented Jul 23, 2019

@squat I remember us having discussed this as well (I thought we opened an issue but I can't find it). Additionally I'd like to see if we could even choose the labels to dedup on at runtime (the use case is thanos receive users that have varying external labels).

@bwplotka
Copy link
Member

AC:

  • Allow deduplication with multiple labels instead of just one (e.g use repeated replica-label flag)
  • Allow specifying deduplication label on run time via extra parameter to query and query_range
  • Document changes.

@bwplotka
Copy link
Member

cc @krasi-georgiev

@krasi-georgiev
Copy link
Contributor

thanks, looking into it.

@bwplotka
Copy link
Member

BTW closing this as it's duplicate of #1174

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants