Skip to content

Add dockerfiles for liberty on UBI 9 minimal#678

Merged
leochr merged 8 commits intovNextfrom
vNext-minimal
Apr 8, 2025
Merged

Add dockerfiles for liberty on UBI 9 minimal#678
leochr merged 8 commits intovNextfrom
vNext-minimal

Conversation

@idlewis
Copy link
Member

@idlewis idlewis commented Mar 28, 2025

For #666

Copy link
Member

@leochr leochr left a comment

Choose a reason for hiding this comment

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

@idlewis Thank you for the PR. As discussed, added some review comments.

# Create a new SCC layer
RUN if [ "$OPENJ9_SCC" = "true" ]; then populate_scc.sh; fi \
&& rm -rf /output/messaging /output/resources/security /logs/* $WLP_OUTPUT_DIR/.classCache \
&& find /opt/ibm/wlp/output ! -perm -g=rwx -print0 | xargs -0 -r chmod g+rwx
Copy link
Member

Choose a reason for hiding this comment

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

-path "*.classCache*" should be added to match IBM Java 8 + UBI 8:

&& find /opt/ibm/wlp/output ! -path "*.classCache*" ! -perm -g=rwx -print0 | xargs -0 -r chmod g+rwx

# limitations under the License.


# TODO
Copy link
Member

Choose a reason for hiding this comment

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

Update this comment to match the commands we run internally so that external users can also build the base image by following the same.

rm -rf /output/workarea /output/logs; \
find /opt/ibm/wlp ! -perm -g=rw -print0 | xargs -r -0 chmod g+rw;

ARG PARENT_IMAGE=icr.io/appcafe/websphere-liberty:kernel-java8-ibmjava-ubi9-minimal
Copy link
Member

Choose a reason for hiding this comment

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

This base image with -ubi9-minimal at the end of the tag name is invalid (including for Semeru 21 based images in production). It should instead be -ubi-minimal.

Copy link
Member

@leochr leochr left a comment

Choose a reason for hiding this comment

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

@idlewis Thanks for the updates. Looks good.

@leochr leochr merged commit a875b91 into vNext Apr 8, 2025
@idlewis idlewis self-assigned this Apr 10, 2025
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