Merge pull request #2656 from stevekuznetsov/skuznets/nothing-to-delete

podman: don't try to delete without volumes

pkg/cluster/internal/providers/podman/provider.go

@@ -165,6 +165,9 @@ func (p *provider) DeleteNodes(n []nodes.Node) error {
 		}
 		nodeVolumes = append(nodeVolumes, volumes...)
 	}
+	if len(nodeVolumes) == 0 {
+		return nil
+	}
 	return deleteVolumes(nodeVolumes)
 }
 

pkg/cluster/internal/providers/podman/util.go

@@ -100,6 +100,10 @@ func getVolumes(label string) ([]string, error) {
 	if err != nil {
 		return nil, err
 	}
+	if string(output) == "" {
+		// no volumes
+		return nil, nil
+	}
 	// Trim away the last `\n`.
 	trimmedOutput := strings.TrimSuffix(string(output), "\n")
 	// Get names of all volumes by splitting via `\n`.