-
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
Revert "cmd/podman: add support for checkpoint images" #16078
Conversation
@rst0git I think we need to revert this merge and rework it on the server side not on the client side. |
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.
LGTM
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhatdan, rst0git The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Tests are very red |
This reverts commit 9c3d8bb. Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan Github is complaining about a DCO? |
This reverts commit bbe1063. Signed-off-by: Daniel J Walsh <[email protected]>
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.
/lgtm
@@ -233,7 +233,7 @@ Pods removed: | |||
`podman kube play --down` will not work with a URL if the YAML file the URL points to | |||
has been changed or altered. | |||
|
|||
|
|||
@@option tls-verify |
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.
@rhatdan do you have any idea why you added this here? It's weird and out of place, I'm 99.99% sure it's an oops. I am going to remove it in my upcoming include-mechanism PR.
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.
SGTM
This reverts commit 9c3d8bb.
This patch needs to be reworked to happen on the server side not on the client side.
Replaces #16068
Does this PR introduce a user-facing change?
[NO NEW TESTS NEEDED]