Increase Timeout in testSLMRetentionAfterRestore (#59979) (#59991)

This test failed by hitting the 10s default busy assert timeout.
Given how involved the retention run is (multiple disk reads, CS updates etc.)
we should have a higher timeout here.

Also, removed the pointless delete call for the snapshot that we just asserted is gone,
 at the end of the test.

Closes #59956
This commit is contained in:
Armin Braun 2020-07-21 18:19:18 +02:00 committed by GitHub
parent dde568caf7
commit 5613e4b00b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -452,14 +452,7 @@ public class SLMSnapshotBlockingIntegTests extends AbstractSnapshotIntegTestCase
// This is what we want to happen // This is what we want to happen
} }
logger.info("--> snapshot [{}] has been deleted", snapshotName); logger.info("--> snapshot [{}] has been deleted", snapshotName);
}); }, 30L, TimeUnit.SECONDS);
// Cancel/delete the snapshot
try {
client().admin().cluster().prepareDeleteSnapshot(REPO, snapshotName).get();
} catch (SnapshotMissingException e) {
// ignore
}
} }
private SnapshotsStatusResponse getSnapshotStatus(String snapshotName) { private SnapshotsStatusResponse getSnapshotStatus(String snapshotName) {