-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
Add '--run' flag to promtool test rules #12206
Add '--run' flag to promtool test rules #12206
Conversation
a1e2a74
to
f163fda
Compare
@dgl are you able to take a look at this? |
Can we find a more intuitive name for the flag? |
maybe --run, like go test? |
Hi @machadovilaca , we looked at this during our bug scrub, would you like to continue working on this pull request and update the flag name to |
@roidelapluie Hi, I've been a little short on time, thanks for the reminder and I'll make sure I take 10min to update the PR this week. |
Signed-off-by: João Vilaça <jvilaca@redhat.com>
f163fda
to
81394ea
Compare
@roidelapluie updated |
@roidelapluie ping |
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.
Thanks! LGTM 💪🏽
Signed-off-by: Julien Pivotto <roidelapluie@o11y.eu>
Thanks! |
When developing unit tests for rules it would be helpful to be able to run only a single group and avoid testing all test groups in a single file.
This PR adds a new
--run
flag topromtool test rules
that allows users to pass multiple regular expressions to filter the test groups that will be run based on their names.