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
Added integration build tag to integration tests. #2164
Conversation
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
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. However, I would like to know @bwplotka opinion, to make sure it doesn't break Thanos tests.
I'm sitting next to @bwplotka and he says "go for it" |
@pstibrany In the meanwhile, could you rebase and add the tag to newly introduced tests? If I'm not mistaken, there should be something. |
Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com>
Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com>
Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com>
Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com>
Done. |
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.
Love it. We would love to actually use same pattern in Thanos.
…ex#2164) * Added integration build tag to integration tests. Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com> * Use tags= syntax Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com> * Fix confused linter complaining about unused symbols. Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com> * Added integration tag. Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com>
What this PR does: This PR adds
integration
tag to Cortex integration tests, so that they don't run by default when doing simplego test ./...
in main Cortex directory.To run integration tests too, one needs to use
-tags=integration
flag togo test
.Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]