Use AND to combine different volume filter keys

Fixes: #26786

Signed-off-by: Šimon Brauner <sbrauner@redhat.com>

Author: simonbrauner

docs/source/markdown/options/filter.volume-ls.md

@@ -5,9 +5,13 @@
 #### **--filter**, **-f**=*filter*
 
 Filter what volumes are shown in the output.
-Multiple filters can be given with multiple uses of the --filter flag.
-Filters with the same key work inclusive, with the only exception being `label`
-which is exclusive. Filters with different keys always work exclusive.
+
+The *filters* argument format is of `key=value`. If there is more than one *filter*, then pass multiple OPTIONS: **--filter** *foo=bar* **--filter** *bif=baz*.
+
+Filters with different `keys` are combined with `AND`.
+Filters with the same `key` are combined with `OR`,
+with the only exceptions being `label` and `label!`,
+which are combined with `AND`.
 
 Volumes can be filtered by the following attributes:
 
@@ -23,3 +27,7 @@ Volumes can be filtered by the following attributes:
 | scope       | Filters volume by scope                                                               |
 | after/since | Filter by volumes created after the given VOLUME (name or tag)                        |
 | until       | Filter by volumes created before given timestamp                                      |
+
+The `label` *filter* accepts two formats. One is the `label`=*key* or `label`=*key*=*value*, which removes volumes with the specified labels. The other format is the `label!`=*key* or `label!`=*key*=*value*, which removes volumes without the specified labels.
+
+The `until` *filter* can be Unix timestamps, date formatted timestamps, or Go duration strings (e.g. 10m, 1h30m) computed relative to the machine's time.

docs/source/markdown/podman-volume-prune.1.md

@@ -27,6 +27,11 @@ Provide filter values.
 
 The *filters* argument format is of `key=value`. If there is more than one *filter*, then pass multiple OPTIONS: **--filter** *foo=bar* **--filter** *bif=baz*.
 
+Filters with different `keys` are combined with `AND`.
+Filters with the same `key` are combined with `OR`,
+with the only exceptions being `label` and `label!`,
+which are combined with `AND`.
+
 Supported filters:
 
 | Filter      | Description                                                                                                |

libpod/runtime_volume.go

@@ -71,7 +71,7 @@ func (r *Runtime) HasVolume(name string) (bool, error) {
 // Volumes retrieves all volumes
 // Filters can be provided which will determine which volumes are included in the
 // output. If multiple filters are used, a volume will be returned if
-// any of the filters are matched
+// all of the filters are matched
 func (r *Runtime) Volumes(filters ...VolumeFilter) ([]*Volume, error) {
 	if !r.valid {
 		return nil, define.ErrRuntimeStopped
@@ -88,9 +88,9 @@ func (r *Runtime) Volumes(filters ...VolumeFilter) ([]*Volume, error) {
 
 	volsFiltered := make([]*Volume, 0, len(vols))
 	for _, vol := range vols {
-		include := false
+		include := true
 		for _, filter := range filters {
-			include = include || filter(vol)
+			include = include && filter(vol)
 		}
 
 		if include {

test/e2e/volume_ls_test.go

@@ -213,6 +213,13 @@ var _ = Describe("Podman volume ls", func() {
 		Expect(session).Should(ExitCleanly())
 		Expect(session.OutputToStringArray()).To(HaveLen(1))
 		Expect(session.OutputToStringArray()[0]).To(Equal(vol3Name))
+
+		// Filters with different keys
+		session = podmanTest.Podman([]string{"volume", "ls", "-q", "--filter", "label=b=c", "--filter", "name=vol1"})
+		session.WaitWithDefaultTimeout()
+		Expect(session).Should(ExitCleanly())
+		Expect(session.OutputToStringArray()).To(HaveLen(1))
+		Expect(session.OutputToStringArray()[0]).To(Equal(vol1Name))
 	})
 
 	It("podman ls volume with --filter since/after", func() {