Skip to content

Commit 447dcaa

Browse files
authored
Disable N-2 BWC tests for non-snapshot builds (#119583)
The tests added in #119468 do not work when executed in the `release-test` pipeline: at some point a 9.0 non-snapshot node tries to join the 8.18.0-SNAPSHOT cluster which is impossible due to a missing `esql.metrics_syntax` feature which is only declared in snapshot builds. Additionaly, the tests rely on unrelease code in 8.x so they won't work until 8.18.0 is released. I don't know how we deal with such issues today, but I'd like to unmute the tests so that they are executed on CI while being disabled in `release-test`. Releates #119550
1 parent bf38719 commit 447dcaa

File tree

2 files changed

+3
-26
lines changed

2 files changed

+3
-26
lines changed

muted-tests.yml

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -236,38 +236,14 @@ tests:
236236
- class: org.elasticsearch.smoketest.MlWithSecurityIT
237237
method: test {yaml=ml/sparse_vector_search/Test sparse_vector search with query vector and pruning config}
238238
issue: https://github.com/elastic/elasticsearch/issues/119548
239-
- class: org.elasticsearch.lucene.RollingUpgradeSearchableSnapshotIndexCompatibilityIT
240-
method: testSearchableSnapshotUpgrade {p0=[9.0.0, 8.18.0, 8.18.0]}
241-
issue: https://github.com/elastic/elasticsearch/issues/119549
242-
- class: org.elasticsearch.lucene.RollingUpgradeSearchableSnapshotIndexCompatibilityIT
243-
method: testMountSearchableSnapshot {p0=[9.0.0, 8.18.0, 8.18.0]}
244-
issue: https://github.com/elastic/elasticsearch/issues/119550
245-
- class: org.elasticsearch.lucene.RollingUpgradeSearchableSnapshotIndexCompatibilityIT
246-
method: testMountSearchableSnapshot {p0=[9.0.0, 9.0.0, 8.18.0]}
247-
issue: https://github.com/elastic/elasticsearch/issues/119551
248239
- class: org.elasticsearch.index.engine.LuceneSyntheticSourceChangesSnapshotTests
249240
method: testSkipNonRootOfNestedDocuments
250241
issue: https://github.com/elastic/elasticsearch/issues/119553
251-
- class: org.elasticsearch.lucene.RollingUpgradeSearchableSnapshotIndexCompatibilityIT
252-
method: testSearchableSnapshotUpgrade {p0=[9.0.0, 9.0.0, 8.18.0]}
253-
issue: https://github.com/elastic/elasticsearch/issues/119560
254-
- class: org.elasticsearch.lucene.RollingUpgradeSearchableSnapshotIndexCompatibilityIT
255-
method: testMountSearchableSnapshot {p0=[9.0.0, 9.0.0, 9.0.0]}
256-
issue: https://github.com/elastic/elasticsearch/issues/119561
257-
- class: org.elasticsearch.lucene.RollingUpgradeSearchableSnapshotIndexCompatibilityIT
258-
method: testSearchableSnapshotUpgrade {p0=[9.0.0, 9.0.0, 9.0.0]}
259-
issue: https://github.com/elastic/elasticsearch/issues/119562
260242
- class: org.elasticsearch.xpack.ml.integration.ForecastIT
261243
method: testOverflowToDisk
262244
issue: https://github.com/elastic/elasticsearch/issues/117740
263245
- class: org.elasticsearch.xpack.security.authc.ldap.MultiGroupMappingIT
264246
issue: https://github.com/elastic/elasticsearch/issues/119599
265-
- class: org.elasticsearch.lucene.FullClusterRestartSearchableSnapshotIndexCompatibilityIT
266-
method: testSearchableSnapshotUpgrade {p0=8.18.0}
267-
issue: https://github.com/elastic/elasticsearch/issues/119631
268-
- class: org.elasticsearch.lucene.FullClusterRestartSearchableSnapshotIndexCompatibilityIT
269-
method: testSearchableSnapshotUpgrade {p0=9.0.0}
270-
issue: https://github.com/elastic/elasticsearch/issues/119632
271247
- class: org.elasticsearch.search.profile.dfs.DfsProfilerIT
272248
method: testProfileDfs
273249
issue: https://github.com/elastic/elasticsearch/issues/119711

qa/lucene-index-compatibility/build.gradle

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,13 @@ buildParams.bwcVersions.withLatestReadOnlyIndexCompatible { bwcVersion ->
1414
tasks.named("javaRestTest").configure {
1515
systemProperty("tests.minimum.index.compatible", bwcVersion)
1616
usesBwcDistribution(bwcVersion)
17-
enabled = true
17+
18+
// Tests rely on unreleased code in 8.18 branch
19+
enabled = buildParams.isSnapshotBuild()
1820
}
1921
}
2022

2123
tasks.withType(Test).configureEach {
2224
// CI doesn't like it when there's multiple clusters running at once
2325
maxParallelForks = 1
2426
}
25-

0 commit comments

Comments
 (0)