Improve logging in e2e tests

test/e2e/main_test.go

@@ -44,6 +44,13 @@ func setupTest(t *testing.T) (string, *framework.TestCtx) {
 		t.Fatalf("could not initialize cluster resources: %v", err)
 	}
 
+	defer func() {
+		showLogsIfTestHasFailed(t, ctx)
+		if t.Failed() && ctx != nil {
+			ctx.Cleanup()
+		}
+	}()
+
 	jenkinsServiceList := &v1alpha2.JenkinsList{
 		TypeMeta: metav1.TypeMeta{
 			Kind:       v1alpha2.Kind,

test/e2e/operator.go

@@ -125,7 +125,7 @@ func printKubernetesPods(t *testing.T, namespace string) {
 	}
 }
 
-func showLogsAndCleanup(t *testing.T, ctx *framework.TestCtx) {
+func showLogsIfTestHasFailed(t *testing.T, ctx *framework.TestCtx) {
 	namespace, err := ctx.GetNamespace()
 	if err != nil {
 		t.Fatalf("Failed to get '%s' namespace", err)
@@ -134,14 +134,22 @@ func showLogsAndCleanup(t *testing.T, ctx *framework.TestCtx) {
 	if t.Failed() {
 		t.Log("Test failed. Bellow here you can check logs:")
 
-		printOperatorLogs(t, namespace)
 		printKubernetesEvents(t, namespace)
 		printKubernetesPods(t, namespace)
+		printOperatorLogs(t, namespace)
+	}
 }
 
-	ctx.Cleanup()
+func showLogsAndCleanup(t *testing.T, ctx *framework.TestCtx) {
-	err = waitUntilNamespaceDestroyed(namespace)
+	namespace, err := ctx.GetNamespace()
 	if err != nil {
+		t.Fatalf("Failed to get '%s' namespace", err)
+	}
+
+	showLogsIfTestHasFailed(t, ctx)
+
+	ctx.Cleanup()
+	if err = waitUntilNamespaceDestroyed(namespace); err != nil {
 		t.Fatalf("Failed to wait for namespace until destroyed '%s'", err)
 	}
 }