fix: only addOrUpdateSubnetQueue if the GatewayType is distributed instead of if it's not centralized (#5493)

josmo · web-flow · commit a29592574d14 · 2025-07-17T13:27:28.000+08:00
Signed-off-by: Joachim Hill-Grannec &lt;joachim.hill@gmail.com&gt;
diff --git a/pkg/controller/node.go b/pkg/controller/node.go
@@ -524,7 +524,7 @@ func (c *Controller) handleUpdateNode(key string) error {
 	}
 
 	for _, cachedSubnet := range subnets {
-		if cachedSubnet.Spec.GatewayType != kubeovnv1.GWCentralizedType {
+		if cachedSubnet.Spec.GatewayType == kubeovnv1.GWDistributedType {
 			// we need to reconcile ovn route for subnets with distributed gateway mode,
 			// since the informer node cache may not be synced before the subnet reconcile triggered by node addition
 			c.addOrUpdateSubnetQueue.Add(cachedSubnet.Name)

Original file line number	Diff line number	Diff line change
`@@ -524,7 +524,7 @@ func (c *Controller) handleUpdateNode(key string) error {`
`524`	`524`	`}`
`525`	`525`
`526`	`526`	`for _, cachedSubnet := range subnets {`
`527`		`- if cachedSubnet.Spec.GatewayType != kubeovnv1.GWCentralizedType {`
	`527`	`+ if cachedSubnet.Spec.GatewayType == kubeovnv1.GWDistributedType {`
`528`	`528`	`// we need to reconcile ovn route for subnets with distributed gateway mode,`
`529`	`529`	`// since the informer node cache may not be synced before the subnet reconcile triggered by node addition`
`530`	`530`	`c.addOrUpdateSubnetQueue.Add(cachedSubnet.Name)`