Skip to content

Commit

Permalink
Fix ClusterDisruptionIT.testAckedIndexing (#53169)
Browse files Browse the repository at this point in the history
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
  • Loading branch information
henningandersen committed Mar 6, 2020
1 parent 23d599d commit 86021ea
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ public void testAckedIndexing() throws Exception {
// 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");

Expand Down

0 comments on commit 86021ea

Please sign in to comment.