From 2e924e4a83f1d157786c4148464cfb0ee8a3e178 Mon Sep 17 00:00:00 2001 From: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Date: Fri, 6 Mar 2020 08:56:27 +0100 Subject: [PATCH] Fix ClusterDisruptionIT.testAckedIndexing (#53169) Use assertBusy when doing reroute after bridged disruption, since it can return non-acked if a node is marked faulty by follower check after disruption ended. Closes #53064 --- .../java/org/elasticsearch/discovery/ClusterDisruptionIT.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/src/test/java/org/elasticsearch/discovery/ClusterDisruptionIT.java b/server/src/test/java/org/elasticsearch/discovery/ClusterDisruptionIT.java index ba0b59c5f16..a2210eb265b 100644 --- a/server/src/test/java/org/elasticsearch/discovery/ClusterDisruptionIT.java +++ b/server/src/test/java/org/elasticsearch/discovery/ClusterDisruptionIT.java @@ -227,7 +227,7 @@ public class ClusterDisruptionIT extends AbstractDisruptionTestCase { // is the super-connected node and recovery source and target are on opposite sides of the bridge if (disruptionScheme instanceof NetworkDisruption && ((NetworkDisruption) disruptionScheme).getDisruptedLinks() instanceof Bridge) { - assertAcked(client().admin().cluster().prepareReroute().setRetryFailed(true)); + assertBusy(() -> assertAcked(client().admin().cluster().prepareReroute().setRetryFailed(true))); } ensureGreen("test");