-
Notifications
You must be signed in to change notification settings - Fork 3k
Support UBI 9 Java 25 images #51677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Support UBI 9 Java 25 images #51677
Conversation
| * Version uses for the native builder image. | ||
| */ | ||
| public static final String NATIVE_BUILDER_VERSION = "jdk-21"; | ||
| public static final String NATIVE_BUILDER_VERSION = "jdk-25"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zakkak I would like your blessing on this one: by adding the full support for Java 25, that means that projects created with Java 25 will target Java 25 and so we need the Mandrel builder to support Java 25. Until now, we always had only one builder for all Java versions and I think that's probably best.
Now the big question: are there any remaining issues using Mandrel for Java 25 or were all the issues sorted out? If they are not sorted out, can we live with them or are they too annoying?
|
/cc @Sanne |
This comment has been minimized.
This comment has been minimized.
|
🎊 PR Preview 9340b2b has been successfully built and deployed to https://quarkus-pr-main-51677-preview.surge.sh/version/main/guides/
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@zakkak so With the last commit I pushed, the build completes but something is incorrect: I wonder if somehow you need to fully embrace I thought about starting the work on #41016 but I'm not sure what's the status of this and if you started working on it already. It's going to be cumbersome so better not do it twice :). |
|
Apparently, the issue is related to #51667 (comment) . |

Fixes #50535