Skip to content
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

Merged
merged 1 commit into from
Jul 1, 2022
Merged

Conversation

yrodiere
Copy link
Member

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.

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.
@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Jun 27, 2022
@yrodiere
Copy link
Member Author

yrodiere commented Jun 27, 2022

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)

@gsmet
Copy link
Member

gsmet commented Jun 27, 2022

I already have a branch with this and @famod has a branch too but it's more complicated than that.

@gsmet
Copy link
Member

gsmet commented Jun 27, 2022

@yrodiere
Copy link
Member Author

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?

@famod
Copy link
Member

famod commented Jun 27, 2022

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.
So we should disable those invoker tests on Java 19+ via Maven profiles.

@famod
Copy link
Member

famod commented Jun 27, 2022

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).
Also, when building with quickly, Gradle modules are the top time consumers.

@famod
Copy link
Member

famod commented Jun 27, 2022

FTR: #24789

@famod
Copy link
Member

famod commented Jun 27, 2022

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).

@yrodiere
Copy link
Member Author

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:

@famod
Copy link
Member

famod commented Jun 27, 2022

IIRC, @glefloch had a look at toolchains quite some time ago. Maybe we should revisit.

@glefloch
Copy link
Member

Yes I think we should otherwise we will be in the same situation for every new Java release

@famod
Copy link
Member

famod commented Jul 1, 2022

Let's just merge this and improve from there.

@famod famod merged commit 3af3a90 into quarkusio:main Jul 1, 2022
@quarkus-bot quarkus-bot bot added this to the 2.11 - main milestone Jul 1, 2022
@yrodiere yrodiere deleted the jdk19 branch August 2, 2022 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants