-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
podman commit: does not create entrypoint? #586
Comments
Podman commit needs to be changed to accept an image config (entrypoint, CMD, etc). We can store the ones the container was created with on the podman side (they're probably in create config anyways) and feed them into commit. |
This should probably happen as part of the commit overhaul to use Buildah code and actually commit diffs (instead of export-import) that is covered by #520 |
Related to #520 |
Resolves: containers#586 and containers#520 Signed-off-by: baude <[email protected]>
When and where should I expect to see this fixed? I'm still seeing the same behavior. Setup:
Curiously, if you |
The code has landed, so if it's not working, we have a new bug with the way we use buildah to commit. |
@TomSweeneyRedHat @baude can one of you perform the comment-0 test on what you believe should be a working system setup, and either reopen this issue or let me know what I've done wrong in my setup? TIA. |
Reopening issue to get confirmation of it working or not. |
Confirmed locally. @TomSweeneyRedHat does Buildah not set image config from the base image? Or are we just calling into it wrong. |
@mheon, looks like it's setting it OK to me unless I messed up the below test. Test Results:
|
@TomSweeneyRedHat Alright, so it's how we're calling into buildah to do a commit, then. |
@baude PTAL |
Vendor in buildah and use as much of commit and bug as possible for podman build and commit. Resolves containers#586 Signed-off-by: baude <[email protected]>
Has anyone tried the three steps in my first comment? Can you get them to work? What am I doing wrong?
buildah-1.00-1526305410.gitce8d467.fc27.x86_64 |
Do those commands work with Docker.
Should do what you want. But if Docker automatically takes the CMD used to create the container then we have a bug. |
Oops - my bad, for not filing it as a docker-compat issue. Yes, docker uses the container's command as an entrypoint:
|
Ok I just looked at this in Docker and the problem is podman does not populating the CMD field on the podman run command. I will open a PR> |
Expected: date output.
podman-0.3.5-1.gitdb6bf9e.fc27.x86_64
The text was updated successfully, but these errors were encountered: