ovs client shouldn't be initialized on DPU Hosts

Signed-off-by: nithyar <[email protected]>

Author: crnithya

go-controller/cmd/ovnkube/ovnkube.go

@@ -538,11 +538,15 @@ func runOvnKube(ctx context.Context, runMode *ovnkubeRunMode, ovnClientset *util
 			// register ovnkube node specific prometheus metrics exported by the node
 			metrics.RegisterNodeMetrics(ctx.Done())
 
-			ovsClient, err = libovsdb.NewOVSClient(ctx.Done())
-			if err != nil {
-				nodeErr = fmt.Errorf("failed to initialize libovsdb vswitchd client: %w", err)
-				return
+			// OVS is not running on dpu-host nodes
+			if config.OvnKubeNode.Mode != types.NodeModeDPUHost {
+				ovsClient, err = libovsdb.NewOVSClient(ctx.Done())
+				if err != nil {
+					nodeErr = fmt.Errorf("failed to initialize libovsdb vswitchd client: %w", err)
+					return
+				}
 			}
+
 			nodeControllerManager, err := controllermanager.NewNodeControllerManager(
 				ovnClientset,
 				watchFactory,

go-controller/pkg/cni/cni.go

@@ -182,8 +182,12 @@ func (pr *PodRequest) cmdAddWithGetCNIResultFunc(
 		if pr.CNIConf.PhysicalNetworkName != "" {
 			netName = pr.CNIConf.PhysicalNetworkName
 		}
-		if err := checkBridgeMapping(ovsClient, pr.CNIConf.Topology, netName); err != nil {
-			return nil, fmt.Errorf("failed bridge mapping validation: %w", err)
+
+		// Skip checking bridge mapping on DPU hosts as OVS is not present
+		if config.OvnKubeNode.Mode != types.NodeModeDPUHost {
+			if err := checkBridgeMapping(ovsClient, pr.CNIConf.Topology, netName); err != nil {
+				return nil, fmt.Errorf("failed bridge mapping validation: %w", err)
+			}
 		}
 
 		response.Result, err = getCNIResultFn(pr, clientset, podInterfaceInfo)