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

Updating UBI images #80

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

arthurdm
Copy link
Contributor

@arthurdm arthurdm commented Nov 4, 2019

In order to be consistent with AdoptOpenJDK's UBI images, as well as the other published IBM Java images, I propose that we do not create user 1001 in the UBI Java layer.

This creates an issue in downstream images, such as Liberty, because we need to switch to root to do things like yum install xyz (creating a larger image size), and requires us to keep different Dockerfiles from our AdoptOpenJDK-based ones (where we need to create user 1001 ourselves).

Signed-off-by: Arthur De Magalhaes <[email protected]>
@arthurdm
Copy link
Contributor Author

arthurdm commented Nov 4, 2019

@dinogun

@dinogun
Copy link
Member

dinogun commented Nov 11, 2019

Hi @arthurdm, I think we added the user to go through the RH certification process and make it available in the RH catalog. Let me check if it is possible without that user.

@arthurdm
Copy link
Contributor Author

arthurdm commented Nov 11, 2019

thanks @dinogun - I didn't know the UBI-min image was on the RH catalog. It would fail the non-root test if 1001 is removed.

It doesn't look like UBI standard (Java) image is in the catalog (as per this page). Do you know if that UBI 8 standard IBM JRE will be uploaded there? If not, then we could update just that Dockerfile, and leave the UBI-min as it is?

Signed-off-by: Arthur De Magalhaes <[email protected]>
@arthurdm
Copy link
Contributor Author

@dinogun - I have updated the PR so that we only change the UBI standard image, which is not published to the RH catalog.

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