Skip to content

Commit

Permalink
Fix: List container with volume filter
Browse files Browse the repository at this point in the history
Modify the condition in line 149 in order to list container by mounting
point.
Closes #16019

Signed-off-by: SamirPS <[email protected]>
  • Loading branch information
SamirPS authored and Samir Akarioh committed Jan 3, 2023
1 parent 2dd9e08 commit e479644
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 e479644

Please sign in to comment.