Skip to content

HDDS-11111. Create ozone-runner base image for Java 11#28

Merged
adoroszlai merged 1 commit intoapache:jdk11from
adoroszlai:HDDS-11111
Sep 7, 2024
Merged

HDDS-11111. Create ozone-runner base image for Java 11#28
adoroszlai merged 1 commit intoapache:jdk11from
adoroszlai:HDDS-11111

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Current ozone-runner comes with JDK 17 (HDDS-8247).

Existing ozone images (up to 1.4.0) ship with some outdated software that we'd like to upgrade (e.g. Robot Framework). However, those Ozone versions do not work with JDK 17.

So we should create ozone-runner image with up-to-date software, but with JDK 11 (like before HDDS-8247), in addition to the newer JDK version(s).

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

How was this patch tested?

Built locally, verified Java version.

$ docker builder build -t apache/ozone-runner:HDDS-11111 .
...

$ docker run -it --rm apache/ozone-runner:HDDS-11111 java -version
To use Ozone please mount ozone folder to /opt/hadoop
openjdk version "11.0.2" 2019-01-15
OpenJDK Runtime Environment 18.9 (build 11.0.2+9)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.2+9, mixed mode)

CI:
https://github.com/adoroszlai/ozone-docker-runner/actions/runs/10421915621/job/28865235225

@adoroszlai adoroszlai self-assigned this Aug 16, 2024
@adoroszlai
Copy link
Contributor Author

@dombizita @smengcl please take a look

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.

lgtm. Thanks @adoroszlai

@adoroszlai adoroszlai merged commit deedbed into apache:jdk11 Sep 7, 2024
@adoroszlai adoroszlai deleted the HDDS-11111 branch September 7, 2024 20:49
@adoroszlai
Copy link
Contributor Author

Thanks @smengcl for the review.

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