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

HDDS-11744. Create ozone-runner version with JDK 21 #34

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

adoroszlai
Copy link
Contributor

@adoroszlai adoroszlai commented Nov 19, 2024

What changes were proposed in this pull request?

Update ozone-runner to Java 21. If we want to keep JDK 17, can be done in a separate branch (jdk17) similarly to JDK 11 (existing jdk11). (So far there is no Ozone release version that works with Java 17 but does not with Java 21, so I don't think it's necessary.)

https://issues.apache.org/jira/browse/HDDS-11744

How was this patch tested?

Image built:
https://github.com/adoroszlai/ozone-docker-runner/actions/runs/11899229183

Ran full Ozone CI with this image and an upgraded third-party dependency (Apache Derby) that requires Java 21.

apache/ozone@master...adoroszlai:ozone:refs/heads/HDDS-10952
https://github.com/adoroszlai/ozone/actions/runs/11900963676

@adoroszlai adoroszlai self-assigned this Nov 19, 2024
@adoroszlai adoroszlai requested a review from smengcl November 19, 2024 08:26
Copy link
Contributor

@smengcl smengcl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1. Thanks @adoroszlai !

Verified that binary URLs and hashes matches build 21.0.2+13 (released on Jan 16, 2024) from OpenJDK Archive.

@adoroszlai adoroszlai merged commit 5d1f1d1 into apache:master Nov 19, 2024
1 check passed
@adoroszlai
Copy link
Contributor Author

Thanks @smengcl for the review.

@adoroszlai adoroszlai deleted the HDDS-11744 branch November 19, 2024 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants