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

Add BASEIMAGE as argument to Containerfile #506

Merged
merged 1 commit into from
May 22, 2024
Merged

Add BASEIMAGE as argument to Containerfile #506

merged 1 commit into from
May 22, 2024

Conversation

enriquebelarte
Copy link
Collaborator

Added BASEIMAGE as ARG so it can be used from build.

Added BASEIMAGE as ARG so it can be used from build.

Signed-off-by: Enrique Belarte Luque <ebelarte@redhat.com>
@Gregory-Pereira
Copy link
Collaborator

Gregory-Pereira commented May 22, 2024

This is what the --from directive does. Ex:

echo "FROM scratch" > Containerfile
podman build --from="registry.redhat.io/ubi9/ubi:latest" . -f Containerfile -t testing
STEP 1/1: FROM registry.redhat.io/ubi9/ubi:latest
COMMIT testing
--> 261134d03e83
Successfully tagged localhost/testing:latest
Successfully tagged registry.redhat.io/ubi9/ubi:latest
Successfully tagged registry.access.redhat.com/ubi9/ubi:latest
261134d03e83034da5dce51c49a45807633c79aa4355c9d57ff7d452a649aa7c

Feel free to read more about this in the podman docs.

If, knowing all this, there is some reason to keep this as a separate build arg instead of from please let me know.

@enriquebelarte
Copy link
Collaborator Author

Konflux CI takes Containerfile and builds it with its own pipeline where unless I'm mistaken we can only use build-args. Hence the change. It is done this way on others.

Copy link
Collaborator

@Gregory-Pereira Gregory-Pereira left a comment

Choose a reason for hiding this comment

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

/lgtm

@rhatdan rhatdan merged commit 8731842 into containers:main May 22, 2024
1 check passed
@rhatdan
Copy link
Member

rhatdan commented May 22, 2024

Are you planning on opening similar PRs for invidia-bootc and intel-bootc?

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