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

ubi9-ibm: unify ADD steps for reef #2176

Merged
merged 1 commit into from
Dec 8, 2023
Merged

ubi9-ibm: unify ADD steps for reef #2176

merged 1 commit into from
Dec 8, 2023

Conversation

ktdreyer
Copy link
Member

@ktdreyer ktdreyer commented Dec 6, 2023

The reef template overrides the contents of the ALL template. As a result, prior to this change, we missed an ADD command in the generated Dockerfile.

Unify and simplify the instructions so we include the proper steps for reef.

The "reef" template overrides the contents of the "ALL" template. As a
result, prior to this change, we missed an ADD command in the generated
Dockerfile.

Unify and simplify the instructions so we include the proper steps for
reef.

Signed-off-by: Ken Dreyer <[email protected]>
@ktdreyer ktdreyer changed the title ubi9-ibm: unify ADD steps ubi9-ibm: unify ADD steps for reef Dec 6, 2023
@bigjust
Copy link
Collaborator

bigjust commented Dec 6, 2023

Ran compose-rhcs with this and it matched the downstream Dockerfile that had both ADD lines, LGTM

@ktdreyer
Copy link
Member Author

ktdreyer commented Dec 7, 2023

@guits would you please merge this? It will avoid a regression.

@guits guits merged commit 02d983d into main Dec 8, 2023
8 checks passed
@guits guits deleted the ibm-ceph-add branch December 8, 2023 13:45
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