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-8247. Bump JDK to 17.0.2 #22

Merged
merged 1 commit into from
Mar 16, 2024
Merged

Conversation

adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Install OpenJDK 17.0.2 in ozone-runner image.

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

How was this patch tested?

Built the image locally, verified java version:

$ docker builder build -t apache/ozone-runner:HDDS-8247 .
...
 => => writing image sha256:abf1973091911ecade064d0e6a8924db5d170879efd079e51c83afc98ac1fbf6                                                                                                                  0.0s
 => => naming to docker.io/apache/ozone-runner:HDDS-8247                                                                                                                                                      0.0s

$ docker run -it --rm apache/ozone-runner:HDDS-8247 java -version
To use Ozone please mount ozone folder to /opt/hadoop
openjdk version "17.0.2" 2022-01-18
OpenJDK Runtime Environment (build 17.0.2+8-86)
OpenJDK 64-Bit Server VM (build 17.0.2+8-86, mixed mode, sharing)

Ran some Ozone acceptance tests with it, verified java version in service logs:

$ grep -m1 'java =' docker-ozone_scm_1.log
STARTUP_MSG:   java = 17.0.2

CI:
https://github.com/adoroszlai/ozone-docker-runner/actions/runs/8306760089

@adoroszlai adoroszlai self-assigned this Mar 16, 2024
@adoroszlai adoroszlai requested a review from smengcl March 16, 2024 09:48
@adoroszlai
Copy link
Contributor Author

When merging this, please make sure to edit the commit message to match PR title (because GitHub uses the message from commit by default if the PR has a single one).

@adoroszlai adoroszlai merged commit 6551048 into apache:master Mar 16, 2024
1 check passed
@adoroszlai adoroszlai deleted the HDDS-8247 branch March 16, 2024 14:03
@adoroszlai
Copy link
Contributor Author

Thanks @smengcl for the review.

@smengcl
Copy link
Contributor

smengcl commented Mar 16, 2024

Thanks @adoroszlai for the patch!

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