Skip to content

[ZEN2] Use Zen2 in REST Tests#36300

Merged
original-brownbear merged 13 commits intoelastic:masterfrom
original-brownbear:zen2-in-rest-tests
Dec 7, 2018
Merged

[ZEN2] Use Zen2 in REST Tests#36300
original-brownbear merged 13 commits intoelastic:masterfrom
original-brownbear:zen2-in-rest-tests

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

No description provided.

@original-brownbear original-brownbear added >test Issues or PRs that are addressing/adding tests WIP v7.0.0 :Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. labels Dec 6, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. I left a few questions.

@original-brownbear
Copy link
Copy Markdown
Contributor Author

@DaveCTurner

I think we should set both properties (for now).

Set to what though? Do you want to set the property in the test settings and revert the code change to prod. code or do you want to actively set a value (1 I guess?) in the prod code when no node count setting but a master list is given?

@original-brownbear
Copy link
Copy Markdown
Contributor Author

Jenkins test this

@ywelsch
Copy link
Copy Markdown
Contributor

ywelsch commented Dec 6, 2018

@elasticmachine retest this please

Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@colings86 colings86 removed the v7.0.0 label Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >test Issues or PRs that are addressing/adding tests v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants