From db13bb17175bfda6d49f205e15dc929bbfb2d15a Mon Sep 17 00:00:00 2001 From: Armin Date: Thu, 31 Jan 2019 10:27:37 +0100 Subject: [PATCH] Don't Assert Ack on when Publish Timeout is 0 in Test * Publish timeout is set to `0` so out of order processing of states on the node can lead to a `false` ack response * See #30672 * Closes #36813 --- .../elasticsearch/indices/state/RareClusterStateIT.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/server/src/test/java/org/elasticsearch/indices/state/RareClusterStateIT.java b/server/src/test/java/org/elasticsearch/indices/state/RareClusterStateIT.java index a34312b847e3b..d2f65d1168da8 100644 --- a/server/src/test/java/org/elasticsearch/indices/state/RareClusterStateIT.java +++ b/server/src/test/java/org/elasticsearch/indices/state/RareClusterStateIT.java @@ -333,12 +333,12 @@ public void testDelayedMappingPropagationOnReplica() throws Exception { // Force allocation of the primary on the master node by first only allocating on the master // and then allowing all nodes so that the replica gets allocated on the other node - assertAcked(prepareCreate("index").setSettings(Settings.builder() + prepareCreate("index").setSettings(Settings.builder() .put(IndexMetaData.SETTING_NUMBER_OF_SHARDS, 1) .put(IndexMetaData.SETTING_NUMBER_OF_REPLICAS, 1) - .put("index.routing.allocation.include._name", master)).get()); - assertAcked(client().admin().indices().prepareUpdateSettings("index").setSettings(Settings.builder() - .put("index.routing.allocation.include._name", "")).get()); + .put("index.routing.allocation.include._name", master)).get(); + client().admin().indices().prepareUpdateSettings("index").setSettings(Settings.builder() + .put("index.routing.allocation.include._name", "")).get(); ensureGreen(); // Check routing tables