Merge pull request #3482 from manugupt1/fix-volume-filter

3452: Filter Volumes by name if any of the names match

Author: djdongjin

cmd/nerdctl/volume/volume_list_test.go

@@ -270,9 +270,6 @@ func TestVolumeLsFilter(t *testing.T) {
 			},
 			{
 				Description: "Retrieving name=volume1 and name=volume2",
-				// FIXME: https://github.com/containerd/nerdctl/issues/3452
-				// Nerdctl filter behavior is broken
-				Require: nerdtest.Docker,
 				Command: func(data test.Data, helpers test.Helpers) test.Command {
 					return helpers.Command("volume", "ls", "--quiet", "--filter", "name="+data.Get("vol1"), "--filter", "name="+data.Get("vol2"))
 				},

pkg/cmd/volume/list.go

@@ -269,15 +269,28 @@ func volumeMatchesFilter(vol native.Volume, labelFilterFuncs []func(*map[string]
 			return false
 		}
 	}
-	for _, nameFilterFunc := range nameFilterFuncs {
-		if !nameFilterFunc(vol.Name) {
-			return false
-		}
+
+	if !anyMatch(vol.Name, nameFilterFuncs) {
+		return false
 	}
+
 	for _, sizeFilterFunc := range sizeFilterFuncs {
 		if !sizeFilterFunc(vol.Size) {
 			return false
 		}
 	}
+
 	return true
 }
+
+func anyMatch[T any](vol T, filters []func(T) bool) bool {
+	if len(filters) == 0 {
+		return true
+	}
+	for _, f := range filters {
+		if f(vol) {
+			return true
+		}
+	}
+	return false
+}