Skip to content

Update container images to use OpenJDK 25#46386

Open
michalvavrik wants to merge 1 commit intokeycloak:mainfrom
michalvavrik:feature/45830-open-jdk-25
Open

Update container images to use OpenJDK 25#46386
michalvavrik wants to merge 1 commit intokeycloak:mainfrom
michalvavrik:feature/45830-open-jdk-25

Conversation

@michalvavrik
Copy link
Member

@michalvavrik michalvavrik commented Feb 16, 2026

This PR is mostly based on vmuzikar@303446b. Additionally, I made few tweaks to documentation and introduce a doc variable for the recommended Java version so limit required changing when migrating the recommended JDK version. I didn't change concepts-memory-and-cpu-sizing.adoc references architecture of some past experiment, so when it says OpenJDK 21, I think it is still true.

@michalvavrik
Copy link
Member Author

hmm, I can see java-25-openjdk-devel.x86_64 on my Fedora, I am going to check what is the issue about.

@michalvavrik
Copy link
Member Author

I think the issue is that UBI8 is used for testing

@michalvavrik michalvavrik force-pushed the feature/45830-open-jdk-25 branch 3 times, most recently from 03dceeb to d9c99a3 Compare February 16, 2026 16:24
@michalvavrik michalvavrik marked this pull request as draft February 16, 2026 16:24
@michalvavrik michalvavrik force-pushed the feature/45830-open-jdk-25 branch from d9c99a3 to 0cafd29 Compare February 16, 2026 16:29
@michalvavrik michalvavrik marked this pull request as ready for review February 16, 2026 16:29
@michalvavrik michalvavrik force-pushed the feature/45830-open-jdk-25 branch from 0cafd29 to 3a1d903 Compare February 16, 2026 16:32
Copy link
Contributor

@vmuzikar vmuzikar left a comment

Choose a reason for hiding this comment

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

@michalvavrik Thank you for the PR.


=== Container Images

The official {project_name} container images for both the server and the operator use OpenJDK {jdk_version_recommended} as the Java runtime environment.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we could be more explicit about the extensions. Also the Operator doesn't probably need to be mentioned here – it's a development guide and we don't support extending the Operator.

Suggested change
The official {project_name} container images for both the server and the operator use OpenJDK {jdk_version_recommended} as the Java runtime environment.
The official {project_name} container image uses OpenJDK {jdk_version_recommended} as the Java runtime environment.
Consider this when writing custom extensions.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done, thanks.

Copy link
Member Author

@michalvavrik michalvavrik Feb 17, 2026

Choose a reason for hiding this comment

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

(well, I didn't add that extra line on purpose, it felt like a single paragraph to me)

Copy link
Member Author

Choose a reason for hiding this comment

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

oh, there was no extra line, sorry, I was in hurry as I need to go for a while and wanted to finish this

This PR is mostly based on vmuzikar@303446b.
Additionally, I made few tweaks to documentation and introduce a doc variable for the recommended Java version so limit required changing when migrating the recommended JDK version.

* Closes: keycloak#45830

Signed-off-by: Michal Vavřík <michal.vavrik@aol.com>
@michalvavrik michalvavrik force-pushed the feature/45830-open-jdk-25 branch from 3a1d903 to c480810 Compare February 17, 2026 13:46
@michalvavrik
Copy link
Member Author

michalvavrik commented Feb 17, 2026

hey @ahus1 I made small change to docs/guides/high-availability/single-cluster/concepts-threads.adoc, can you please check it's alright with you or ask someone from your team to check?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update container images to use OpenJDK 25

2 participants