Skip to content

Commit

Permalink
Support zen2
Browse files Browse the repository at this point in the history
Starting with [1], `cluster.initial_master_nodes` is required. This
can point to a list of node names, but can also support a list of
<IP:PORT> or just <IP>  addresses.

Fix Elasticsearch setup by Rally on the `master` branch by supporting
the new cluster initial master nodes property.

Also remove `discovery.zen.minimum_master_nodes` which is not required
any more.

[1]
elastic/elasticsearch@6e6e63d

Relates #10
  • Loading branch information
dliappis authored Dec 11, 2018
1 parent 62b2d44 commit 6acc645
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions cars/v1/vanilla/templates/config/elasticsearch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,17 +70,13 @@ transport.tcp.port: {{transport_port}}
#
{%- if all_node_ips %}
discovery.zen.ping.unicast.hosts: {{ all_node_ips }}
# Prevent split brain by specifying the initial master nodes.
cluster.initial_master_nodes: {{ all_node_ips }}
{%- else %}
#discovery.zen.ping.unicast.hosts: ["host1", "host2"]
#cluster.initial_master_nodes: ["node-name1", "node-name2"]
{%- endif %}
#
# Prevent the "split brain" by configuring the majority of nodes (total number of nodes / 2 + 1):
#
{%- if minimum_master_nodes %}
discovery.zen.minimum_master_nodes: {{ minimum_master_nodes }}
{%- else %}
#discovery.zen.minimum_master_nodes: 3
{%- endif %}
#
# For more information, see the documentation at:
# <http://www.elastic.co/guide/en/elasticsearch/reference/current/modules-discovery.html>
Expand Down

0 comments on commit 6acc645

Please sign in to comment.