Don't Consistency Check Broken Repository in Test (#43499)

original-brownbear · web-flow · commit 32904522e9db · 2019-06-22T16:50:08.000+02:00
* Missed this one in #42189 and it randomly runs into a situation where the broken mock repo is broken such that we can't get to a consistent end state via a delete * Closes #43498
diff --git a/server/src/test/java/org/elasticsearch/snapshots/SharedClusterSnapshotRestoreIT.java b/server/src/test/java/org/elasticsearch/snapshots/SharedClusterSnapshotRestoreIT.java
@@ -842,8 +842,9 @@ public void testSnapshotFileFailureDuringSnapshot() throws InterruptedException
         }
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/43498")
     public void testDataFileFailureDuringSnapshot() throws Exception {
+        disableRepoConsistencyCheck("This test intentionally leaves a broken repository");
+
         Client client = client();
         logger.info("-->  creating repository");
         assertAcked(client.admin().cluster().preparePutRepository("test-repo")

Original file line number	Diff line number	Diff line change
`@@ -842,8 +842,9 @@ public void testSnapshotFileFailureDuringSnapshot() throws InterruptedException`
`842`	`842`	`}`
`843`	`843`	`}`
`844`	`844`
`845`		`- @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/43498")`
`846`	`845`	`public void testDataFileFailureDuringSnapshot() throws Exception {`
	`846`	`+ disableRepoConsistencyCheck("This test intentionally leaves a broken repository");`
	`847`	`+`
`847`	`848`	`Client client = client();`
`848`	`849`	`logger.info("--> creating repository");`
`849`	`850`	`assertAcked(client.admin().cluster().preparePutRepository("test-repo")`