Skip to content

[RORDEV-1941] Do not apply JDK patch, remove ROR <=1.64 fallbacks#88

Merged
mgoworko merged 3 commits into
masterfrom
RORDEV-1941
May 16, 2026
Merged

[RORDEV-1941] Do not apply JDK patch, remove ROR <=1.64 fallbacks#88
mgoworko merged 3 commits into
masterfrom
RORDEV-1941

Conversation

@mgoworko
Copy link
Copy Markdown
Collaborator

@mgoworko mgoworko commented May 9, 2026

The JDK patch related to https://bugs.openjdk.org/browse/JDK-8287073 no longer needs to be applied since ROR 1.69. Removing also patching-related args for ROR<1.64

Summary by CodeRabbit

  • Chores

    • Reduced environment bootstrap time by pre-pulling runtime images and removing the prior local Elasticsearch JDK-patch build.
    • Simplified image selection to derive runtime images from configured repository/version values.
    • Reordered startup to create the local cluster earlier and apply required kernel settings to nodes before continuing.
  • Bug Fixes

    • Consolidated patch-consent setting to a single environment variable for clearer configuration.

Review Change Stack

@coderabbitai

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as resolved.

@mgoworko mgoworko changed the base branch from master to develop May 9, 2026 21:29
@mgoworko mgoworko changed the title [RORDEV-1941] Do not apply JDK patch for ROR >= 1.69 [RORDEV-1941] Do not apply JDK patch, remove ROR <=1.64 fallbacks May 15, 2026
@mgoworko mgoworko changed the base branch from develop to master May 15, 2026 18:46
@mgoworko mgoworko requested a review from coutoPL May 15, 2026 20:52
Copy link
Copy Markdown
Collaborator

@coutoPL coutoPL left a comment

Choose a reason for hiding this comment

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

LGTM

@mgoworko mgoworko merged commit afca53b into master May 16, 2026
82 checks passed
@coutoPL coutoPL deleted the RORDEV-1941 branch May 18, 2026 09:04
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