SNAPSHOT+TESTS: Stabilize SnapshotDisruptionIT

* Ensure retry by busy assert on SnapshotMissingException * Closes elastic#36739
original-brownbear · Jan 10, 2019 · 7990faf · 7990faf
1 parent d625b79
commit 7990faf
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/server/src/test/java/org/elasticsearch/discovery/SnapshotDisruptionIT.java b/server/src/test/java/org/elasticsearch/discovery/SnapshotDisruptionIT.java
@@ -75,7 +75,6 @@ protected Settings nodeSettings(int nodeOrdinal) {
             .build();
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/36779")
     public void testDisruptionOnSnapshotInitialization() throws Exception {
         final String idxName = "test";
         final List<String> allMasterEligibleNodes = internalCluster().startMasterOnlyNodes(3);
@@ -180,7 +179,11 @@ public void clusterChanged(ClusterChangedEvent event) {
 
         logger.info("--> verify that snapshot eventually will be created due to retries");
         assertBusy(() -> {
-            assertSnapshotExists("test-repo", "test-snap-2");
+            try {
+                assertSnapshotExists("test-repo", "test-snap-2");
+            } catch (SnapshotMissingException ex) {
+                throw new AssertionError(ex);
+            }
         }, 1, TimeUnit.MINUTES);
     }