fix underlay e2e testing (#1929)

kubeovn · Sep 28, 2022 · 73fde2c · 73fde2c
1 parent 38956b6
commit 73fde2c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/pkg/daemon/init_linux.go b/pkg/daemon/init_linux.go
@@ -1,6 +1,8 @@
 package daemon
 
 import (
+	"syscall"
+
 	"k8s.io/klog/v2"
 
 	"github.com/Wifx/gonetworkmanager"
@@ -105,7 +107,7 @@ func changeProvideNicName(current, target string) (bool, error) {
 				continue
 			}
 			if route.Scope == scope {
-				if err = netlink.RouteReplace(&route); err != nil {
+				if err = netlink.RouteReplace(&route); err != nil && err != syscall.EEXIST {
 					klog.Errorf("failed to replace route %s: %v", route.String(), err)
 					return false, err
 				}

diff --git a/test/e2e/underlay/underlay.go b/test/e2e/underlay/underlay.go
@@ -197,7 +197,10 @@ var _ = Describe("[Underlay]", func() {
 				Expect(err).NotTo(HaveOccurred())
 			}
 
+			By("wait provider network to be ready")
 			time.Sleep(3 * time.Second)
+			err = f.WaitProviderNetworkReady(ProviderNetwork)
+			Expect(err).NotTo(HaveOccurred())
 
 			By("validate provider network")
 			pn, err := f.OvnClientSet.KubeovnV1().ProviderNetworks().Get(context.Background(), ProviderNetwork, metav1.GetOptions{})
@@ -222,6 +225,11 @@ var _ = Describe("[Underlay]", func() {
 			newPn.Spec.ExcludeNodes = nil
 			_, err = f.OvnClientSet.KubeovnV1().ProviderNetworks().Update(context.Background(), newPn, metav1.UpdateOptions{})
 			Expect(err).NotTo(HaveOccurred())
+
+			By("wait provider network to be ready")
+			time.Sleep(3 * time.Second)
+			err = f.WaitProviderNetworkReady(ProviderNetwork)
+			Expect(err).NotTo(HaveOccurred())
 		})
 	})