Do not rely on CurrentSpecReport.Failed in AfterSuite

Signed-off-by: Kyle Squizzato <[email protected]>
k0rdent · Sep 26, 2024 · 6150edd · 6150edd
1 parent b5e0279
commit 6150edd
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/test/e2e/e2e_suite_test.go b/test/e2e/e2e_suite_test.go
@@ -72,11 +72,9 @@ var _ = BeforeSuite(func() {
 
 var _ = AfterSuite(func() {
 	if !noCleanup() {
-		if CurrentSpecReport().Failed() {
-			By("collecting failure logs from local controllers")
-			kc := kubeclient.NewFromLocal(internalutils.DefaultSystemNamespace)
-			collectLogArtifacts(kc, "")
-		}
+		By("collecting logs from local controllers")
+		kc := kubeclient.NewFromLocal(internalutils.DefaultSystemNamespace)
+		collectLogArtifacts(kc, "")
 
 		By("removing the controller-manager")
 		cmd := exec.Command("make", "dev-destroy")