Skip to content

Commit d0b1a4a

Browse files
arteamjfreden
authored andcommitted
Revert "[test] Dynamically pick up the upper bound snapshot index version (elastic#1…" (elastic#115827)
This reverts commit 32dee6a.
1 parent 9fcc49d commit d0b1a4a

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)