Skip to content

Commit

Permalink
Increase Timeout in testSnapshotRestore (#60532) (#60538)
Browse files Browse the repository at this point in the history
It seems this test only fails with `FsRepository` and mostly just barely times out (takes just a little over 30s to go green).
I think just increasing the timeout should be fine as a fix here as it's a little interesting to check larger amounts of
data in this test generally speaking.

Closes #39299
  • Loading branch information
original-brownbear authored Jul 31, 2020
1 parent f1d4fd8 commit 8c7eae1
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
import org.elasticsearch.common.bytes.BytesArray;
import org.elasticsearch.common.io.Streams;
import org.elasticsearch.common.settings.Settings;
import org.elasticsearch.common.unit.TimeValue;
import org.elasticsearch.repositories.IndexId;
import org.elasticsearch.repositories.RepositoriesService;
import org.elasticsearch.repositories.Repository;
Expand Down Expand Up @@ -321,7 +322,8 @@ public void testSnapshotAndRestore() throws Exception {
logger.info("--> restore all indices from the snapshot");
assertSuccessfulRestore(client().admin().cluster().prepareRestoreSnapshot(repoName, snapshotName).setWaitForCompletion(true));

ensureGreen();
// higher timeout since we can have quite a few shards and a little more data here
ensureGreen(TimeValue.timeValueSeconds(120));

for (int i = 0; i < indexCount; i++) {
assertHitCount(client().prepareSearch(indexNames[i]).setSize(0).get(), docCounts[i]);
Expand Down

0 comments on commit 8c7eae1

Please sign in to comment.