Skip to content

Commit

Permalink
[ILM][TEST] increase assertBusy timeout (elastic#36864)
Browse files Browse the repository at this point in the history
the testFullPolicy and testMoveToRolloverStep tests
are very important tests, but they sometimes timeout
beyond the default 10sec wait for shrink to occur.
This commit increases one of the assertBusys to
20 seconds
  • Loading branch information
talevy committed Feb 4, 2019
1 parent cb901bb commit 9f38313
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@
import java.util.List;
import java.util.Locale;
import java.util.Map;
import java.util.concurrent.TimeUnit;
import java.util.function.Supplier;

import static java.util.Collections.singletonMap;
Expand Down Expand Up @@ -99,7 +100,7 @@ public void testFullPolicy() throws Exception {
// asserts that rollover was called
assertBusy(() -> assertTrue(indexExists(secondIndex)));
// asserts that shrink deleted the original index
assertBusy(() -> assertFalse(indexExists(originalIndex)));
assertBusy(() -> assertFalse(indexExists(originalIndex)), 20, TimeUnit.SECONDS);
// asserts that the delete phase completed for the managed shrunken index
assertBusy(() -> assertFalse(indexExists(shrunkenOriginalIndex)));
}
Expand Down Expand Up @@ -177,7 +178,7 @@ public void testMoveToRolloverStep() throws Exception {
// asserts that rollover was called
assertBusy(() -> assertTrue(indexExists(secondIndex)));
// asserts that shrink deleted the original index
assertBusy(() -> assertFalse(indexExists(originalIndex)));
assertBusy(() -> assertFalse(indexExists(originalIndex)), 20, TimeUnit.SECONDS);
// asserts that the delete phase completed for the managed shrunken index
assertBusy(() -> assertFalse(indexExists(shrunkenOriginalIndex)));
}
Expand Down

0 comments on commit 9f38313

Please sign in to comment.