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

Migrate container image/Dockerfile from UBI8 to UBI9 #695

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

patchkez
Copy link
Contributor

@patchkez patchkez commented Sep 19, 2024

This PR should replace of using UBI8 images with UBI9.
I was able to build new image with:

podman build . -f ./src/main/docker/Dockerfile.jvm -t test_build_policies_ui_backend --ulimit=nofile=4096

I confirmed that curl exists in the final image as I had to remove it from dockerfile due to installation conflict:

podman run --rm -it --entrypoint=sh localhost/test_build_policies_ui_backend:latest
sh-5.1$ 
sh-5.1$ curl --version
curl 7.76.1 (x86_64-redhat-linux-gnu) libcurl/7.76.1 OpenSSL/3.0.7 zlib/1.2.11 nghttp2/1.43.0
Release-Date: 2021-04-14
Protocols: file ftp ftps http https 
Features: alt-svc AsynchDNS GSS-API HTTP2 HTTPS-proxy IPv6 Kerberos Largefile libz NTLM SPNEGO SSL UnixSockets

RHINENG-12536

@vkrizan vkrizan self-requested a review September 20, 2024 11:30
@vkrizan
Copy link
Contributor

vkrizan commented Sep 20, 2024

@patchkez can you please also change the GitHub Action that does an image check? https://github.com/RedHatInsights/policies-ui-backend/blob/master/.github/workflows/checkupdate.yml#L23

@patchkez
Copy link
Contributor Author

@patchkez can you please also change the GitHub Action that does an image check? https://github.com/RedHatInsights/policies-ui-backend/blob/master/.github/workflows/checkupdate.yml#L23

done

Copy link
Contributor

@vkrizan vkrizan left a comment

Choose a reason for hiding this comment

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

Thank you

@vkrizan vkrizan merged commit b7bade3 into RedHatInsights:master Sep 20, 2024
4 checks passed
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