Skip to content

Revert addition of logstash.jdk.force setting - #19375

Merged
andsel merged 3 commits into
elastic:8.19from
donoghuc:revert-new-setting
Aug 4, 2026
Merged

Revert addition of logstash.jdk.force setting#19375
andsel merged 3 commits into
elastic:8.19from
donoghuc:revert-new-setting

Conversation

@donoghuc

@donoghuc donoghuc commented Aug 3, 2026

Copy link
Copy Markdown
Member

Release notes

[rn:skip]

What does this PR do?

Reverts addition of logstash.jdk.force and all references to it while we figure out the best ergonomics and deprecation path. The behavior should remain the same as in 8.19.19 whereby if a custom jdk 17 is detected a deprecation will be logged and logstash will continue to run normally.

Reverts #19311 #19361

@donoghuc
donoghuc requested a review from andsel August 3, 2026 21:25
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • run exhaustive tests : Run the exhaustive tests Buildkite pipeline.

@donoghuc

donoghuc commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

run exhaustive tests

@donoghuc

donoghuc commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

@andsel andsel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@andsel
andsel merged commit 5ec0e02 into elastic:8.19 Aug 4, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants