Skip to content

Commit afab439

Browse files
committed
Set minimum_master_nodes to all nodes for REST tests (#27344)
PR #26911 set minimum_master_nodes from number_of_nodes to (number_of_nodes / 2) + 1 in our REST tests. This has led to test failures (see #27233) as the REST tests only configure the first node in its unicast.hosts pinging list (see explanation here: #27233 (comment)). Until we have a proper fix for this, I'm reverting the change in #26911.
1 parent e938c12 commit afab439

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

buildSrc/src/main/groovy/org/elasticsearch/gradle/test/ClusterConfiguration.groovy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,10 @@ class ClusterConfiguration {
6464

6565
/**
6666
* Configuration of the setting <tt>discovery.zen.minimum_master_nodes</tt> on the nodes.
67-
* In case of more than one node, this defaults to (number of nodes / 2) + 1
67+
* In case of more than one node, this defaults to the number of nodes
6868
*/
6969
@Input
70-
Closure<Integer> minimumMasterNodes = { getNumNodes() > 1 ? getNumNodes().intdiv(2) + 1 : -1 }
70+
Closure<Integer> minimumMasterNodes = { getNumNodes() > 1 ? getNumNodes() : -1 }
7171

7272
@Input
7373
String jvmArgs = "-Xms" + System.getProperty('tests.heap.size', '512m') +

0 commit comments

Comments
 (0)