Do not process a namespace again if it's already processed

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
rancher · Sep 4, 2021 · 189e740 · 189e740
1 parent d6e566c
commit 189e740
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/pkg/manager/cluster.go b/pkg/manager/cluster.go
@@ -85,9 +85,17 @@ func (c *Cluster) generateSupportBundleYAMLs(yamlsDir string, errLog io.Writer)
 	// Namespaced scope: all resources
 	namespaces := []string{"default", "kube-system", "cattle-system"}
 	namespaces = append(namespaces, c.sbm.Namespaces...)
+
+	done := make(map[string]struct{})
 	for _, namespace := range namespaces {
+		if _, ok := done[namespace]; ok {
+			continue
+		}
+
 		namespacedDir := filepath.Join(yamlsDir, "namespaced", namespace)
 		c.generateDiscoveredNamespacedYAMLs(namespace, namespacedDir, errLog)
+
+		done[namespace] = struct{}{}
 	}
 }