Skip to content

Commit ff4f9d7

Browse files
authored
Re-enable geoip FullClusterRestartIT (#120966) (#120976)
1 parent e807b44 commit ff4f9d7

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

modules/ingest-geoip/qa/full-cluster-restart/build.gradle

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,7 @@ dependencies {
2121
javaRestTestImplementation(testArtifact(project(":qa:full-cluster-restart"), "javaRestTest"))
2222
}
2323

24-
assert Version.fromString(VersionProperties.getVersions().get("elasticsearch")).getMajor() == 8 :
25-
"If we are targeting a branch other than 8, we should enable migration tests"
26-
27-
// once we are ready to test migrations from 8.x to 9.x, we can set the compatible version to 8.0.0
28-
// see https://github.com/elastic/elasticsearch/pull/93666
29-
buildParams.bwcVersions.withWireCompatible(v -> v.before("7.0.0")) { bwcVersion, baseName ->
24+
buildParams.bwcVersions.withWireCompatible(v -> v.before("8.0.0")) { bwcVersion, baseName ->
3025
tasks.register(bwcTaskName(bwcVersion), StandaloneRestIntegTestTask) {
3126
usesBwcDistribution(bwcVersion)
3227
systemProperty("tests.old_cluster_version", bwcVersion)

modules/ingest-geoip/qa/full-cluster-restart/src/javaRestTest/java/org/elasticsearch/ingest/geoip/FullClusterRestartIT.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,8 @@ public class FullClusterRestartIT extends ParameterizedFullClusterRestartTestCas
4242
.distribution(DistributionType.DEFAULT)
4343
.version(getOldClusterTestVersion())
4444
.nodes(2)
45-
.setting("indices.memory.shard_inactive_time", "60m")
46-
.setting("xpack.security.enabled", "false")
4745
.setting("ingest.geoip.downloader.endpoint", () -> fixture.getAddress(), s -> useFixture)
46+
.setting("xpack.security.enabled", "false")
4847
.feature(FeatureFlag.TIME_SERIES_MODE)
4948
.build();
5049

@@ -93,7 +92,7 @@ public void testGeoIpSystemFeaturesMigration() throws Exception {
9392
Request migrateSystemFeatures = new Request("POST", "/_migration/system_features");
9493
assertOK(client().performRequest(migrateSystemFeatures));
9594

96-
assertBusy(() -> testCatIndices(".geoip_databases-reindexed-for-8", "my-index-00001"));
95+
assertBusy(() -> testCatIndices(".geoip_databases-reindexed-for-9", "my-index-00001"));
9796
assertBusy(() -> testIndexGeoDoc());
9897

9998
Request disableDownloader = new Request("PUT", "/_cluster/settings");

0 commit comments

Comments
 (0)