-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Tools: Add flag --selector.relabel-config-file
/ --selector.relabel-config
/ --max-time
/ --min-time
to filter served blocks.
#4453
Tools: Add flag --selector.relabel-config-file
/ --selector.relabel-config
/ --max-time
/ --min-time
to filter served blocks.
#4453
Conversation
--selector.relabel-config-file
/ --selector.relabel-config
/ --max-time
/ --min-time
to filter served blocks.
49d6996
to
873fbb6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with only one nit. Thanks!
8f41d70
to
e07be14
Compare
Signed-off-by: hanjm <hanjinming@outlook.com>
Signed-off-by: hanjm <hanjinming@outlook.com>
Signed-off-by: hanjm <hanjinming@outlook.com>
Signed-off-by: hanjm <hanjinming@outlook.com>
Signed-off-by: hanjm <hanjinming@outlook.com>
e07be14
to
452f4eb
Compare
Signed-off-by: hanjm <hanjinming@outlook.com>
I have add e2e test, @yeya24 could you help me review again? thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work! Just two small nits.
Signed-off-by: hanjm <hanjinming@outlook.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
fix #4452
Signed-off-by: hanjm hanjinming@outlook.com
Changes
min-time
max-time
selector.relabel-config
flag tothanos tools bucket web
.Verification
e2e test.
run with new flag, it works.