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

Move to UBI8 parent image #168

Merged
merged 1 commit into from
May 8, 2023
Merged

Conversation

gsaslis
Copy link
Contributor

@gsaslis gsaslis commented Apr 27, 2023

This will help with producing a product build for limitador, and makes many things easier such as rebuilds on Common Vulnerability and Exposure (CVE) patches, (e.g. Container Health Index (CHI) - the ‘A’ / ‘B’ / ‘C’ grades shown on Red Hat Container Catalog (RHCC) ), cross-product dependencies, etc.

Notes:

  • slightly different package names in ubi8, but this shouldn't be introducing anything new.

This will help with producing a product build for limitador, and makes many things easier such as rebuilds on Common Vulnerability and Exposure (CVE) patches, (e.g. Container Health Index (CHI) - the ‘A’ / ‘B’ / ‘C’ grades shown on Red Hat Container Catalog (RHCC) ), cross-product dependencies, etc.

Notes:
* slightly different package names in ubi8, but this shouldn't be introducing anything new.
Dockerfile Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
@gsaslis
Copy link
Contributor Author

gsaslis commented May 8, 2023

guess this is ready to merge ?

Copy link
Contributor

@eguzki eguzki left a comment

Choose a reason for hiding this comment

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

As @alexsnaps pointed out offline quite rightly, the runtime base image should be aligned with the building base image to avoid issues with dynamic linking. Most concerning one: openssl

@alexsnaps alexsnaps merged commit fb68c4a into Kuadrant:main May 8, 2023
@alexsnaps
Copy link
Member

Thanks for the PR @gsaslis

@gsaslis gsaslis deleted the feat/ubi8_parent_image branch May 8, 2023 14:53
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.

3 participants