TESTS+DISTR.: Fix testIndexCheckOnStartup Flake (#33349)

original-brownbear · web-flow · commit 42424aff21e1 · 2018-09-03T17:06:12.000+02:00
* Ignore all `RuntimeException` since random file corruption triggers other RTE in addition to the randomly caught one * closes #33345
diff --git a/server/src/test/java/org/elasticsearch/index/shard/IndexShardTests.java b/server/src/test/java/org/elasticsearch/index/shard/IndexShardTests.java
@@ -2596,7 +2596,6 @@ public void testReadSnapshotConcurrently() throws IOException, InterruptedExcept
         closeShards(newShard);
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/33345")
     public void testIndexCheckOnStartup() throws Exception {
         final IndexShard indexShard = newStartedShard(true);
 
@@ -2650,7 +2649,7 @@ public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IO
         try {
             closeShards(corruptedShard);
         } catch (RuntimeException e) {
-            assertThat(e.getMessage(), equalTo("CheckIndex failed"));
+            // Ignored because corrupted shard can throw various exceptions on close
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -2596,7 +2596,6 @@ public void testReadSnapshotConcurrently() throws IOException, InterruptedExcept`
`2596`	`2596`	`closeShards(newShard);`
`2597`	`2597`	`}`
`2598`	`2598`
`2599`		`- @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/33345")`
`2600`	`2599`	`public void testIndexCheckOnStartup() throws Exception {`
`2601`	`2600`	`final IndexShard indexShard = newStartedShard(true);`
`2602`	`2601`
`@@ -2650,7 +2649,7 @@ public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IO`
`2650`	`2649`	`try {`
`2651`	`2650`	`closeShards(corruptedShard);`
`2652`	`2651`	`} catch (RuntimeException e) {`
`2653`		`- assertThat(e.getMessage(), equalTo("CheckIndex failed"));`
	`2652`	`+ // Ignored because corrupted shard can throw various exceptions on close`
`2654`	`2653`	`}`
`2655`	`2654`	`}`
`2656`	`2655`