Skip to content

Commit

Permalink
Merge pull request #16934 from SamirPS/fix-volume
Browse files Browse the repository at this point in the history
Fix: List container with volume filter
  • Loading branch information
openshift-merge-robot authored Jan 3, 2023
2 parents 2dcf6b1 + e479644 commit 28d04bc
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/domain/filters/containers.go
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ func GenerateContainerFilterFuncs(filter string, filterValues []string, r *libpo
if dest != "" && (mount.Source == source && mount.Destination == dest) {
return true
}
if dest == "" && mount.Source == source {
if dest == "" && mount.Destination == source {
return true
}
}
Expand Down
16 changes: 16 additions & 0 deletions test/system/160-volumes.bats
Original file line number Diff line number Diff line change
Expand Up @@ -509,4 +509,20 @@ EOF
is "$output" "" "Should print no output"
}

@test "podman ps -f" {
vol1="/v1_$(random_string)"
run_podman run -d --rm --volume ${PODMAN_TMPDIR}:$vol1 $IMAGE top
cid=$output

run_podman ps --noheading --no-trunc -q -f volume=$vol1
is "$output" "$cid" "Should find container by volume"

run_podman ps --noheading --no-trunc -q --filter volume=/NoSuchVolume
is "$output" "" "ps --filter volume=/NoSuchVolume"

# Clean up
run_podman rm -f -t 0 -a
}


# vim: filetype=sh

0 comments on commit 28d04bc

Please sign in to comment.