Skip to content

Commit

Permalink
Fix SLMSnapshotBlockingIntegTests (elastic#47841)
Browse files Browse the repository at this point in the history
One of the tests in this suit stops a master node,
plus we're doing other node starts in this suit.
=> the internal test cluster should be TEST and not `SUITE`
scoped to avoid random failures like the one in elastic#47834

Closes elastic#47834
  • Loading branch information
original-brownbear committed Oct 10, 2019
1 parent 17433e7 commit 4bcfdce
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@
*/
@TestLogging(value = "org.elasticsearch.snapshots.mockstore:DEBUG",
reason = "https://github.com/elastic/elasticsearch/issues/46508")
@ESIntegTestCase.ClusterScope(scope = ESIntegTestCase.Scope.TEST)
public class SLMSnapshotBlockingIntegTests extends ESIntegTestCase {

private static final String REPO = "repo-id";
Expand Down Expand Up @@ -167,7 +168,6 @@ public void testSnapshotInProgress() throws Exception {
}
}

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/47834")
public void testRetentionWhileSnapshotInProgress() throws Exception {
final String indexName = "test";
final String policyId = "slm-policy";
Expand Down

0 comments on commit 4bcfdce

Please sign in to comment.