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

Missing slim fixes to ubi, ubi-minimal, clefos #561

Merged
merged 8 commits into from
May 19, 2021

Conversation

srbala
Copy link
Contributor

@srbala srbala commented Apr 29, 2021

@dinogun Fix contains necessary changes requried resolve the slim package issues in ubi and others

This PR Does not contains the multi stage build, whichI indicated in discussion. Would like that to be add in this PR or send as another one. This PR is failrly straight forwrd, Multi stage build requires additional unit testing.

@srbala
Copy link
Contributor Author

srbala commented Apr 29, 2021

@dinogun @sxa @karianna this PR fixes three issues, #339, #478 and #550.

@karianna karianna linked an issue May 5, 2021 that may be closed by this pull request
@karianna karianna added this to the May 2021 milestone May 5, 2021
@srbala
Copy link
Contributor Author

srbala commented May 6, 2021

@karianna CI build failre related to alpine image, not related to changes in this pull request. Possibliy related to another issue of runtime library.

Copy link
Member

@karianna karianna left a comment

Choose a reason for hiding this comment

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

LGTM

@karianna karianna requested a review from gdams May 7, 2021 17:08
dockerfile_functions.sh Outdated Show resolved Hide resolved
dockerfile_functions.sh Outdated Show resolved Hide resolved
dockerfile_functions.sh Outdated Show resolved Hide resolved
Signed-off-by: Bala Raman <[email protected]>
@srbala
Copy link
Contributor Author

srbala commented May 15, 2021

@dinogun fixes ready for review

@karianna karianna requested a review from dinogun May 17, 2021 11:18
dockerfile_functions.sh Outdated Show resolved Hide resolved
dockerfile_functions.sh Outdated Show resolved Hide resolved
@srbala
Copy link
Contributor Author

srbala commented May 18, 2021

@dinogun proposed changes completed, ready for review

dockerfile_functions.sh Outdated Show resolved Hide resolved
@dinogun
Copy link
Collaborator

dinogun commented May 19, 2021

Removed slim images for JRE from the config as it is not currently supported, the tests are now passing.

Copy link
Collaborator

@dinogun dinogun left a comment

Choose a reason for hiding this comment

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

LGTM

@dinogun dinogun merged commit 3f377e0 into AdoptOpenJDK:master May 19, 2021
@srbala srbala deleted the fix-478 branch May 19, 2021 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UBI Slim image missing call to slim-java.sh
3 participants