test/framework: fix disable metrics collection should not disable wat…

…ching deployment logs Signed-off-by: Stefan Büringer [email protected]
kubernetes-sigs · Aug 9, 2021 · ccaebbf · ccaebbf
1 parent 92e3867
commit ccaebbf
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/test/framework/clusterctl/clusterctl_helpers.go b/test/framework/clusterctl/clusterctl_helpers.go
@@ -109,15 +109,14 @@ func InitManagementClusterAndWatchControllerLogs(ctx context.Context, input Init
 			LogPath:    filepath.Join(input.LogFolder, "controllers"),
 		})
 
-		if input.DisableMetricsCollection {
-			return
+		if !input.DisableMetricsCollection {
+			framework.WatchPodMetrics(ctx, framework.WatchPodMetricsInput{
+				GetLister:   client,
+				ClientSet:   input.ClusterProxy.GetClientSet(),
+				Deployment:  deployment,
+				MetricsPath: filepath.Join(input.LogFolder, "controllers"),
+			})
 		}
-		framework.WatchPodMetrics(ctx, framework.WatchPodMetricsInput{
-			GetLister:   client,
-			ClientSet:   input.ClusterProxy.GetClientSet(),
-			Deployment:  deployment,
-			MetricsPath: filepath.Join(input.LogFolder, "controllers"),
-		})
 	}
 }