Skip to content

Commit 480cd13

Browse files
authored
Revert "[test] Dynamically pick up the upper bound snapshot index version (#1…"
This reverts commit 32dee6a.
1 parent d0f71fc commit 480cd13

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

test/framework/src/main/java/org/elasticsearch/snapshots/AbstractSnapshotIntegTestCase.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import org.elasticsearch.common.unit.ByteSizeUnit;
3535
import org.elasticsearch.common.util.concurrent.EsExecutors;
3636
import org.elasticsearch.core.Nullable;
37+
import org.elasticsearch.core.UpdateForV9;
3738
import org.elasticsearch.index.IndexVersion;
3839
import org.elasticsearch.index.IndexVersions;
3940
import org.elasticsearch.plugins.Plugin;
@@ -365,9 +366,15 @@ protected static Settings.Builder indexSettingsNoReplicas(int shards) {
365366
/**
366367
* Randomly write an empty snapshot of an older version to an empty repository to simulate an older repository metadata format.
367368
*/
369+
@UpdateForV9(owner = UpdateForV9.Owner.DISTRIBUTED_COORDINATION)
370+
// This used to pick an index version from 7.0.0 to 8.9.0. The minimum now is 8.0.0 but it's not clear what the upper range should be
368371
protected void maybeInitWithOldSnapshotVersion(String repoName, Path repoPath) throws Exception {
369372
if (randomBoolean() && randomBoolean()) {
370-
initWithSnapshotVersion(repoName, repoPath, IndexVersionUtils.randomVersion());
373+
initWithSnapshotVersion(
374+
repoName,
375+
repoPath,
376+
IndexVersionUtils.randomVersionBetween(random(), IndexVersions.MINIMUM_COMPATIBLE, IndexVersions.V_8_9_0)
377+
);
371378
}
372379
}
373380

0 commit comments

Comments
 (0)