Skip to content

Commit ae9f4df

Browse files
Don't Assert Ack on when Publish Timeout is 0 in Test (elastic#38077)
* Publish timeout is set to `0` so out of order processing of states on the node can lead to a `false` ack response * See elastic#30672 * Closes elastic#36813
1 parent 9f026bb commit ae9f4df

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

server/src/test/java/org/elasticsearch/indices/state/RareClusterStateIT.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -333,12 +333,12 @@ public void testDelayedMappingPropagationOnReplica() throws Exception {
333333

334334
// Force allocation of the primary on the master node by first only allocating on the master
335335
// and then allowing all nodes so that the replica gets allocated on the other node
336-
assertAcked(prepareCreate("index").setSettings(Settings.builder()
336+
prepareCreate("index").setSettings(Settings.builder()
337337
.put(IndexMetaData.SETTING_NUMBER_OF_SHARDS, 1)
338338
.put(IndexMetaData.SETTING_NUMBER_OF_REPLICAS, 1)
339-
.put("index.routing.allocation.include._name", master)).get());
340-
assertAcked(client().admin().indices().prepareUpdateSettings("index").setSettings(Settings.builder()
341-
.put("index.routing.allocation.include._name", "")).get());
339+
.put("index.routing.allocation.include._name", master)).get();
340+
client().admin().indices().prepareUpdateSettings("index").setSettings(Settings.builder()
341+
.put("index.routing.allocation.include._name", "")).get();
342342
ensureGreen();
343343

344344
// Check routing tables

0 commit comments

Comments
 (0)