Increase Timeout in testSLMRetentionAfterRestore (#59979)

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 17:46:54 +02:00 committed by GitHub
parent 08de504b44
commit eaf34bb57d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

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