-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Upgrade to JDK 19 for JDK early-access build #26374
Conversation
Also switch to oracle/setup-java by default, because that's the easiest way to retrieve the latest binaries for EA releases. It only supports Oracle builds though, so we keep actions/setup-java for people who need to launch this workflow manually.
Note it's about time, since EA builds of JDK 20 are starting to get published... :) (JDK 19 is still EA at the moment, though) |
I already have a branch with this and @famod has a branch too but it's more complicated than that. |
I'm sure upgrading Quarkus will require more work, but there's no point keeping a workflow with JDK 18 EA running. Let's just merge this and disable the workflow until someone can get it to work? |
The strategy up to now was to aggressively disable the bits that are failing (if there is no upstream fix yet), but I haven't found the time to finish this for 19-ea. |
Btw, a general flag to disable Gradle and/or Kotlin bits might come in handy as those never seem to work on EAs (maybe shortly before GA, if you're lucky). |
FTR: #24789 |
Having said all that, I think this change makes sense for now but maybe we should also remove the schedule until someone found the time to drive it home (but retain the ability to run manually). |
Regarding Gradle, you should definitely look into toolchains. We're using them in Hibernate ORM / Reactive and have been upgrading to newer JDKs (including 19) without any problem for a while. It requires that you install both JDK 17 (for Gradle) and 19 (for your build/tests), though. See:
And ideally you'd use a different JDK to build the main code and to build/run the tests, but that's more complicated, so I wouldn't recommend it as a first step. See: |
IIRC, @glefloch had a look at toolchains quite some time ago. Maybe we should revisit. |
Yes I think we should otherwise we will be in the same situation for every new Java release |
Let's just merge this and improve from there. |
Also switch to oracle/setup-java by default, because that's the easiest
way to retrieve the latest binaries for EA releases.
It only supports Oracle builds though, so we keep actions/setup-java for
people who need to launch this workflow manually.