Skip to content

Commit a008345

Browse files
committed
L2 and L3 UDN should reconfigure reroute policies when join IP changes
Signed-off-by: Tim Rozet <[email protected]>
1 parent bf5b8d4 commit a008345

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

go-controller/pkg/ovn/secondary_layer2_network_controller.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,8 @@ func (h *secondaryLayer2NetworkControllerEventHandler) UpdateResource(oldObj, ne
180180
hostCIDRsChanged(oldNode, newNode) ||
181181
nodeGatewayMTUSupportChanged(oldNode, newNode)
182182
_, syncRerouteFailed := h.oc.syncEIPNodeRerouteFailed.Load(newNode.Name)
183-
shouldSyncReroute := syncRerouteFailed || util.NodeHostCIDRsAnnotationChanged(oldNode, newNode)
183+
shouldSyncReroute := syncRerouteFailed || util.NodeHostCIDRsAnnotationChanged(oldNode, newNode) ||
184+
joinCIDRChanged(oldNode, newNode, h.oc.GetNetworkName())
184185
nodeSyncsParam = &nodeSyncs{
185186
syncMgmtPort: shouldSyncMgmtPort,
186187
syncGw: shouldSyncGW,

go-controller/pkg/ovn/secondary_layer3_network_controller.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,8 @@ func (h *secondaryLayer3NetworkControllerEventHandler) UpdateResource(oldObj, ne
186186
hostCIDRsChanged(oldNode, newNode) ||
187187
nodeGatewayMTUSupportChanged(oldNode, newNode)
188188
_, failed = h.oc.syncEIPNodeRerouteFailed.Load(newNode.Name)
189-
syncReroute := failed || util.NodeHostCIDRsAnnotationChanged(oldNode, newNode)
189+
syncReroute := failed || util.NodeHostCIDRsAnnotationChanged(oldNode, newNode) ||
190+
joinCIDRChanged(oldNode, newNode, h.oc.GetNetworkName())
190191
nodeSyncsParam = &nodeSyncs{
191192
syncNode: nodeSync,
192193
syncClusterRouterPort: clusterRtrSync,

0 commit comments

Comments
 (0)