-
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
api,images: add support for LookupManifest
to Image removal
REST API
#14783
api,images: add support for LookupManifest
to Image removal
REST API
#14783
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flouthoc 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 |
cceaeaf
to
1672024
Compare
1672024
to
72c347d
Compare
ImagesBatchRemoval and ImageRemoval now honors and accepts `LookupManifest` parameter which further tells libimage to resolve to manifest list if it exists instead of actual image. Following PR also makes `podman-remote manifest rm` functional which was broken till now. Closes: containers#14763 Signed-off-by: Aditya R <[email protected]>
72c347d
to
7bbfb3e
Compare
Restarted failing test looked unrelated to the PR in the first instance. @vrothberg PTAL |
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
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
ImagesBatchRemoval and ImageRemoval now honors and accepts
LookupManifest
parameter which further tells libimage to resolve tomanifest list if it exists instead of actual image.
Following PR also makes
podman-remote manifest rm
functional which wasbroken till now.
Closes: #14763
Does this PR introduce a user-facing change?