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

remote: fix podman-remote container checkpoint --compress #21056

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ttsuuubasa
Copy link
Contributor

Fix podman container checkpoint --compress to work in remote environment.

Currently, whichever a compression type (zstd, gzip, none) is selected, the exported file is compressed with "none" as the following.
This PR enable the command to compress the exported file with the selected compression type.

I have added the function for checking a compression type of the exported file into e2e test.

# podman-remote run --name test -d alpine top
28de2ef85c7edf565bfa24ef2b5bf1eb9be4b90ae1cc0f97ea995cfbaefd93f0

# podman-remote container checkpoint -e test-zstd.tar.gz --compress zstd test
test

# file test-zstd.tar.gz
test-zstd.tar.gz: POSIX tar archive <- "none"

# podman-remote container checkpoint -e test-gzip.tar.gz --compress gzip test
test

# file test-gzip.tar.gz
test-gzip.tar.gz: POSIX tar archive <- "none"

# podman-remote container checkpoint -e test-none.tar.gz --compress none test
test

# file test-none.tar.gz
test-none.tar.gz: POSIX tar archive <- "none"

Does this PR introduce a user-facing change?

The remote Podman client's `podman container checkpoint --compress` option work now.

Copy link
Contributor

openshift-ci bot commented Dec 19, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fj-tsubasa
Once this PR has been reviewed and has the lgtm label, please assign saschagrunert for approval. For more information see the Kubernetes Code Review Process.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions github-actions bot added the kind/api-change Change to remote API; merits scrutiny label Dec 19, 2023
@rhatdan
Copy link
Member

rhatdan commented Dec 19, 2023

Thanks, This looks good.

I am not sure what is going on with the tests, but did you run

make generate-bindings
And then commit the changes.

@ttsuuubasa ttsuuubasa force-pushed the podman-remote-compression branch 3 times, most recently from 82bf6d5 to c835737 Compare December 28, 2023 07:56
Fix `podman container checkpoint --compress` to work in remote
environment.

Signed-off-by: Tsubasa Watanabe <[email protected]>
@ttsuuubasa ttsuuubasa force-pushed the podman-remote-compression branch from c835737 to 87d3ae2 Compare December 28, 2023 10:41
@ttsuuubasa
Copy link
Contributor Author

@rhatdan

I've done "make generate-bindings" and committed the changes.
Could you please review that?

@ttsuuubasa
Copy link
Contributor Author

@rhatdan

Could you please comment somewhat?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/api-change Change to remote API; merits scrutiny release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants