when restart deployment kube-ovn-controller the kubectl ko log loss (#…

…2508)

* when restart deployment kube-ovn-controller the kubectl ko log loss

* fix

test/e2e/framework/deployment.go

@@ -3,6 +3,7 @@ package framework
 import (
 	"context"
 	"fmt"
+	"os/exec"
 	"time"
 
 	appsv1 "k8s.io/api/apps/v1"
@@ -118,3 +119,15 @@ func MakeDeployment(name string, replicas int32, podLabels, podAnnotations map[s
 	deploy.Spec.Template.Annotations = podAnnotations
 	return deploy
 }
+
+func RestartSystemDeployment(name string, enableLog bool) {
+	if enableLog {
+		output, err := exec.Command("bash", "-c", fmt.Sprintf("kubectl ko log kube-ovn %s all", name)).CombinedOutput()
+		framework.ExpectNoError(err, fmt.Sprintf("dump all %s log failed", name))
+		framework.Logf(string(output))
+	}
+
+	restartCmd := fmt.Sprintf("kubectl rollout restart deployment %s -n kube-system", name)
+	_, err := exec.Command("bash", "-c", restartCmd).CombinedOutput()
+	framework.ExpectNoError(err, fmt.Sprintf("restart %s failed", name))
+}

test/e2e/kube-ovn/underlay/underlay.go

@@ -543,9 +543,7 @@ var _ = framework.SerialDescribe("[group:underlay]", func() {
 		checkU2OItems(true, subnet, underlayPod, overlayPod)
 
 		ginkgo.By("step4: check if kube-ovn-controller restart")
-		restartCmd := "kubectl rollout restart deployment kube-ovn-controller -n kube-system"
-		_, err = exec.Command("bash", "-c", restartCmd).CombinedOutput()
-		framework.ExpectNoError(err, "restart kube-ovn-controller")
+		framework.RestartSystemDeployment("kube-ovn-controller", true)
 		checkU2OItems(true, subnet, underlayPod, overlayPod)
 
 		ginkgo.By("step5: Disable u2o check after restart kube-controller")