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

Build against Java 21 #2929

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Build against Java 21 #2929

wants to merge 9 commits into from

Conversation

sstone
Copy link
Member

@sstone sstone commented Oct 14, 2024

These are small, non-functional changes that are needed to target Java 21. The only real issue was the kanela agent, the version we used is not compatible with JDK21.
Eclair will require a Java runtime that is compatible with Java 21 and will no longer run on JRE 11 or JRE 17: DO NOT MERGE this PR until production environments have been updated with a Java 21 compatible JRE !!

.mvn/wrapper/maven-wrapper.properties Show resolved Hide resolved
.mvn/wrapper/maven-wrapper.properties Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
mvnw Outdated Show resolved Hide resolved
@sstone sstone changed the title Changes needed to be compatible with JDK21 Changes needed to be compatible with a Java 21 runtime Oct 15, 2024
@sstone sstone changed the title Changes needed to be compatible with a Java 21 runtime Build against Java 21 Oct 21, 2024
Previous version was not compatible with JDK21, see kamon-io/kanela#150.
This will make it easier to control which version of maven is used to build eclair, which in turns makes deterministic builds easier, as well as
using recent compiler options (to target newer JDKs for example).
For example, even recent versions of Github runner images use an old verion of maven and there is no easy way to upgrade.
We now use multiarch (amd64/arm64) base images.
Eclair now targets Java 21 and will require a compatible Java Runtime Environment.
It will no longer work on JRE 11 and JRE 17.
@sstone sstone marked this pull request as ready for review December 12, 2024 09:26
@sstone
Copy link
Member Author

sstone commented Dec 12, 2024

Our test and production images have been updated and this PR is now safe to merge.

@sstone sstone requested a review from t-bast December 12, 2024 09:51
.mvn/wrapper/maven-wrapper.properties Show resolved Hide resolved
docs/release-notes/eclair-vnext.md Show resolved Hide resolved
.mvn/wrapper/maven-wrapper.properties Outdated Show resolved Hide resolved
@sstone sstone requested a review from pm47 December 12, 2024 13:43
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.

2 participants