Skip to content

Commit

Permalink
Rename cluster settings following new 2.x convention
Browse files Browse the repository at this point in the history
Previously the setting was `cluster.initial_master_nodes` and has since
changed to `cluster.initial_cluster_manager_nodes`.  This was causing
issues due to a bug in the fallback method that has been fixed in
OpenSearch, but getting ahead of this change before its needed.

See related opensearch-project/OpenSearch#2779

Signed-off-by: Peter Nied <petern@amazon.com>
  • Loading branch information
peternied committed Apr 6, 2022
1 parent 2972bd1 commit 7379f76
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,7 @@ private Settings.Builder getMinimumNonSecurityNodeSettingsBuilder(final int node
.put("path.logs", "./target/data/"+clustername+"/logs")
.put("node.max_local_storage_nodes", nodeCount)
//.put("discovery.zen.minimum_master_nodes", minMasterNodes(masterTcpPorts.size()))
.putList("cluster.initial_master_nodes", masterTcpPorts.stream().map(s->"127.0.0.1:"+s).collect(Collectors.toList()))
.putList("cluster.initial_cluster_manager_nodes", masterTcpPorts.stream().map(s->"127.0.0.1:"+s).collect(Collectors.toList()))
//.put("discovery.zen.no_master_block", "all")
//.put("discovery.zen.fd.ping_timeout", "5s")
.put("discovery.initial_state_timeout","8s")
Expand Down
2 changes: 1 addition & 1 deletion tools/install_demo_configuration.sh
Original file line number Diff line number Diff line change
Expand Up @@ -382,7 +382,7 @@ else
if [ "$cluster_mode" == 1 ]; then
echo "network.host: 0.0.0.0" | $SUDO_CMD tee -a "$OPENSEARCH_CONF_FILE" > /dev/null
echo "node.name: smoketestnode" | $SUDO_CMD tee -a "$OPENSEARCH_CONF_FILE" > /dev/null
echo "cluster.initial_master_nodes: smoketestnode" | $SUDO_CMD tee -a "$OPENSEARCH_CONF_FILE" > /dev/null
echo "cluster.initial_cluster_manager_nodes: smoketestnode" | $SUDO_CMD tee -a "$OPENSEARCH_CONF_FILE" > /dev/null
fi
fi

Expand Down

0 comments on commit 7379f76

Please sign in to comment.