Skip to content

Conversation

@andrershov
Copy link
Contributor

Today InternalTestClusterTests is still using zen1.
This commit fixes it.
Two types of changes were required:

  1. Explicitly pass file discovery host provider setting. It's done in ESIntegTestCase as a part of the Zen2 feature and should be done here as well.
  2. For the test, that uses autoManageMinMasterNodes = false perform cluster bootstrap. Currently, I'm using INITIAL_MASTER_NODE_COUNT_SETTING setting that will be replaced with INITIAL_MASTER_NODES_SETTING once Replace InMemoryPersistedState with GatewayMetaState in CoordinatorTests #36897 is merged.

@andrershov andrershov added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. labels Dec 24, 2018
@andrershov andrershov requested a review from ywelsch December 24, 2018 13:42
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

2 + (masterNodes ? InternalTestCluster.DEFAULT_HIGH_NUM_MASTER_NODES : 0) + maxNumDataNodes + numClientNodes)
.put(NetworkModule.TRANSPORT_TYPE_KEY, getTestTransportType())
.put(USE_ZEN2.getKey(), false) // full cluster restarts not yet supported
.putList(DISCOVERY_HOSTS_PROVIDER_SETTING.getKey(), "file")
Copy link
Contributor

Choose a reason for hiding this comment

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

also set .putList(SettingsBasedHostsProvider.DISCOVERY_ZEN_PING_UNICAST_HOSTS_SETTING.getKey()) // empty list disables a port scan for other nodes?

@andrershov
Copy link
Contributor Author

run default distro tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants