Skip to content

[9.2] Warn on API key version mismatch (#143127)#143301

Merged
elasticsearchmachine merged 1 commit intoelastic:9.2from
ebarlas:backport/9.2/pr-143127
Feb 27, 2026
Merged

[9.2] Warn on API key version mismatch (#143127)#143301
elasticsearchmachine merged 1 commit intoelastic:9.2from
ebarlas:backport/9.2/pr-143127

Conversation

@ebarlas
Copy link
Copy Markdown
Contributor

@ebarlas ebarlas commented Feb 27, 2026

Backports the following commits to 9.2:

During a rolling upgrade, an older node could encounter an
API key stamped with a newer version than it recognizes.
This hasn't been an issue in practice since API key versions
haven't changed, but the assert in maybeBuildIndexRequest
treated it as impossible, firing an AssertionError that
killed the JVM in CI where assertions are enabled.

Replace the assert with a WARN-level log. The update
proceeds as before, rewriting the doc with the local
node's target version. This matches existing production
behavior (where assertions are disabled) but adds
observability via the warning.
@ebarlas ebarlas added :Security/Security Security issues without another label >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Security Meta label for security team labels Feb 27, 2026
@elasticsearchmachine elasticsearchmachine merged commit d237f85 into elastic:9.2 Feb 27, 2026
41 checks passed
@ebarlas ebarlas deleted the backport/9.2/pr-143127 branch February 27, 2026 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Security/Security Security issues without another label Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v9.2.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants