-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
feat(libpod): support kube play tar content-type #24015
feat(libpod): support kube play tar content-type #24015
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fixomatic-ctrl The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8ec742c
to
8d8b4d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does sound like a good approach to me, you should update the swagger doc for this endpoint to specify the accept content-tpye and what format it expects in the tar
cf19d57
to
6ee082d
Compare
/retest |
@fixomatic-ctrl: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Wonder if this is worth implementing for the CLI side of things for Podman Desktop. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code wise LGTM
It would be nice to have an actual API level test here with a tarball, please see test/apiv2 for such test. There are already some libpod/build tests there that show how it should be done.
6ee082d
to
8d0e48c
Compare
Ephemeral COPR build failed. @containers/packit-build please check. |
@Luap99 while making the tests, I noticed that if we already build the image(s), we do not have any way to force the re-build. It would re-use the existing image. Therefore I added a query parameter |
Signed-off-by: fixomatic-ctrl <[email protected]>
8d0e48c
to
019fda7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks this looks mostly good but you have to use $IMAGE in the tests to make them work in CI reliably
Signed-off-by: fixomatic-ctrl <[email protected]>
Part of #14527
Does this PR introduce a user-facing change?
Testing with a tar file
Create a test directory with the following structure
play.yaml
example/foobar/Containerfile
FROM docker.io/library/alpine:latest
make binaries
Create a podman system service
./bin/podman system service --time=0 tcp://localhost:8888
Create a tar file
example.tar
with our example contentcd ./example && tar cvf example.tar *
Testing backward compatibilty
Let's check we did not break the basic support
play.yaml
Change to
./bin/podman kube play example/play.yaml