-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Test Java 21 on CI #36284
Test Java 21 on CI #36284
Conversation
FYI, I'm making progress on this and testing in my fork for now as I iterate slowly. |
5f5bd50
to
c7b1b47
Compare
I'm not exactly sure what's wrong with the JDK 21 build but I end up with:
I'm investigating what's going on but I'm not entirely sure if I will be able to get the info, especially if the problem is with the build generating too many files... |
🎊 PR Preview b2179b8 has been successfully built and deployed to https://quarkus-pr-main-36284-preview.surge.sh/version/main/guides/ |
bac4c63
to
bad5f4f
Compare
We are now using Java 21 so we don't need the flag anymore.
This should now be ready. The tests are passing and I fixed/disabled the failing tests. Let's see what CI has to say. |
This comment has been minimized.
This comment has been minimized.
This test is not compatible with Java 21. See jakartaee/cdi-tck#485
Failing Jobs - Building 1632b31
Full information is available in the Build summary check run. Failures⚙️ JVM Tests - JDK 17 Windows #- Failing: extensions/vertx-http/deployment
! Skipped: devtools/cli extensions/agroal/deployment extensions/amazon-lambda-http/deployment and 364 more 📦 extensions/vertx-http/deployment✖
⚙️ Maven Tests - JDK 11 Windows #- Failing: integration-tests/maven
📦 integration-tests/maven✖
✖
✖
✖
⚙️ Native Tests - Messaging1 #- Failing: integration-tests/kafka-oauth-keycloak
📦 integration-tests/kafka-oauth-keycloak✖ |
@snazy trying this crazy stuff in my fork for now :).