Skip to content

Fix index version constant#139496

Merged
jfreden merged 3 commits intoelastic:9.1from
jordan-powers:fix-index-version-constant-91
Dec 15, 2025
Merged

Fix index version constant#139496
jfreden merged 3 commits intoelastic:9.1from
jordan-powers:fix-index-version-constant-91

Conversation

@jordan-powers
Copy link
Contributor

We need to use a patch version, not a full version here. 9.1.8 was the previous commit (50f58de), so this hasn't been released yet - we can modify this in time for 9.1.9.

This is the same problem as fixed in #139361 on 9.2.

Discovered by #139482.

@jordan-powers jordan-powers added blocker :Core/Infra/Core Core issues without another label >refactoring :Security/Security Security issues without another label stateful Marking issues only relevant for stateful releases v9.1.8 v9.1.9 labels Dec 14, 2025
@elasticsearchmachine elasticsearchmachine added v9.1.10 Team:Core/Infra Meta label for core/infra team Team:Security Meta label for security team labels Dec 14, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jfreden
Copy link
Contributor

jfreden commented Dec 15, 2025

Some upgrade tests are failing with:

org.elasticsearch.transport.RemoteTransportException: [test-cluster-2][127.0.0.1:34763][internal:cluster/coordination/join]
Caused by: java.lang.IllegalStateException: index [.inference/b-HfCvLdRY27B9W3DI4Zpw] version not supported: 9.1.9 maximum compatible index version is: 9.1.9

@jfreden
Copy link
Contributor

jfreden commented Dec 15, 2025

One of the tests that fail is:

./gradlew ":x-pack:plugin:inference:qa:rolling-upgrade:bcUpgradeTest" -Dtests.class="org.elasticsearch.xpack.application.OpenAiServiceUpgradeIT" -Dtests.method="testOpenAiEmbeddings {upgradedNodes=1}" -Dtests.seed=A7F17D71A1B794CB -Dtests.bwc.main.version=9.1.9-SNAPSHOT -Dtests.bwc.refspec.main=c6ce60656077305524f7950a6b01f0d442d3f575 -Dtests.locale=ar-Arab-EG -Dtests.timezone=Etc/GMT+2 -Druntime.java=25

So it's doing an upgrade from 9.1.0-SNAPSHOT (current 9.1 "main", see this), to the version in the PR, but the index version in the PR code is older than on 9.1, because the fix in this PR is not merged yet. Therefore this error is expected and can be ignored.

@jfreden jfreden merged commit ba75b76 into elastic:9.1 Dec 15, 2025
38 of 41 checks passed
@jordan-powers jordan-powers deleted the fix-index-version-constant-91 branch January 15, 2026 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocker :Core/Infra/Core Core issues without another label >refactoring :Security/Security Security issues without another label stateful Marking issues only relevant for stateful releases Team:Core/Infra Meta label for core/infra team Team:Security Meta label for security team v9.1.8 v9.1.9 v9.1.10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments