-
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
fix != filter in volume prune #17483
Conversation
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 LGTM, thanks!
Is there an issue for this fix?
@containers/podman-maintainers 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.
PR LGTM but I think there is a issue in CI unrelated to this PR. Maybe something related to CI itself cc @cevich
Maybe rebasing on the latest HEAD will help |
1da95c3
to
5278f4b
Compare
@vrothberg Thanks for the review. I have rebased and updated the PR -- hopefully it will resolve the issue. |
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
/hold
Tests aren´t happy yet |
Yes, it's my tests. Taking a look |
5278f4b
to
65f5bfb
Compare
65f5bfb
to
b70e7d1
Compare
I found the error in CI, yep just needed to rebase to fix that. |
Signed-off-by: vyasgun <[email protected]>
b70e7d1
to
33a9187
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flouthoc, vrothberg, vyasgun 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 |
/hold cancel |
Signed-off-by: vyasgun [email protected]
Fixes #17051
Does this PR introduce a user-facing change?