From 6acc645df9d32aad18b610b710bc54e2f73d9468 Mon Sep 17 00:00:00 2001 From: Dimitrios Liappis Date: Tue, 11 Dec 2018 14:06:29 +0200 Subject: [PATCH] Support zen2 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 or just 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] https://github.com/elastic/elasticsearch/commit/6e6e63d01da8fab85d3a69656416c39bfd8c9139 Relates #10 --- cars/v1/vanilla/templates/config/elasticsearch.yml | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/cars/v1/vanilla/templates/config/elasticsearch.yml b/cars/v1/vanilla/templates/config/elasticsearch.yml index a453443..41e8f02 100644 --- a/cars/v1/vanilla/templates/config/elasticsearch.yml +++ b/cars/v1/vanilla/templates/config/elasticsearch.yml @@ -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: #