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
Tests: Add back tests.cluster #9916
Conversation
Thanks a lot for looking into this @rjernst ! Just one comment: I wonder if it makes sense to have an external cluster with scope TEST. given that the system property value will be the same and we don't recreate anything per test... it looks like it is going to be the same as SUITE, no? Maybe we should only do this for SUITE scope? maybe even barf if we try to run something with scope TEST against the external cluster? Thoughts? |
@@ -1621,8 +1621,32 @@ protected Settings nodeSettings(int nodeOrdinal) { | |||
protected Settings transportClientSettings() { | |||
return ImmutableSettings.EMPTY; | |||
} | |||
|
|||
private ExternalTestCluster buildExternalCluster(String clusterAddresses) { | |||
logger.info("BUILDING EXTERNAL CLUSTER"); |
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.
why is this log uppercase? :)
@javanna I pushed a new commit that addresses your comments. The log message was unintentional. |
LGTM thanks @rjernst |
50c3774
to
6ab717f
Compare
6ab717f
to
1a78074
Compare
This setting is used by the release script to run rest tests against the version being released. It used to work only for tests using the global cluster. Now it supercedes both SUITE and TEST scope test clusters. closes elastic#9916
1a78074
to
d59f5a3
Compare
This setting is used by the release script to run rest tests against the version being released. It used to work only for tests using the global cluster. Now it supercedes both SUITE and TEST scope test clusters. closes #9916
This setting is used by the release script to run rest tests against
the version being released. It used to work only for tests using
the global cluster. Now it supercedes both SUITE and TEST scope
test clusters.