[SecondaryNetwork] Fix agent crash if host interface is already attac…

…hed to OVS bridge PrepareHostInterfaceConnection should return the bridged interface name if the host interface is already attached to the OVS bridge. Signed-off-by: Xu Liu <[email protected]>
antrea-io · Sep 12, 2024 · 2815123 · 2815123
1 parent afdd261
commit 2815123
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/agent/util/net_linux.go b/pkg/agent/util/net_linux.go
@@ -419,12 +419,12 @@ func PrepareHostInterfaceConnection(
 	// If the port already exists, just return.
 	if ofPort, err := bridge.GetOFPort(bridgedName, false); err == nil {
 		klog.InfoS("Port already exists, skip the configuration", "port", bridgedName, "ofPort", ofPort)
-		return "", true, nil
+		return bridgedName, true, nil
 	}
 
 	iface, ifaceIPs, ifaceRoutes, err := GetInterfaceConfig(ifaceName)
 	if err != nil {
-		return "", false, nil
+		return "", false, err
 	}
 
 	if err = RenameInterface(ifaceName, bridgedName); err != nil {