Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update policy route when change from ecmp to active-standby #2716

Merged
merged 1 commit into from
May 3, 2023

Conversation

hongzhen-ma
Copy link
Collaborator

@hongzhen-ma hongzhen-ma commented Apr 26, 2023

What type of this PR

  • Bug fixes

集中式子网由 ecmp 模式,切换为主备模式,需要删除非activeGateway 网关节点的策略路由

Which issue(s) this PR fixes:

Fixes #(issue-number)

WHAT

🤖 Generated by Copilot at cb282d6

This pull request enhances the support for ECMP routing for centralized subnets in kube-ovn. It modifies the checkPolicyRouteExistForNode function in node.go and the routing logic in subnet.go to handle different ECMP scenarios.

🤖 Generated by Copilot at cb282d6

Sing, O Muse, of the clever coder who devised
A better way to route the packets of the cloud,
With checkPolicyRouteExistForNode he revised
The function to support ECMP, of which he was proud.

HOW

🤖 Generated by Copilot at cb282d6

  • Modify checkPolicyRouteExistForNode to return next hops and node name-ip map for ECMP support (link)
  • Modify checkPolicyRouteExistForNode to check policy route existence based on ECMP configuration (link)
  • Delete ActivateGateway field from subnet status when ECMP is enabled in reconcileOvnRoute (link)
  • Add active-backup policy route for centralized subnet when ECMP is disabled in reconcileOvnRoute (link)

pkg/controller/node.go Show resolved Hide resolved
@hongzhen-ma hongzhen-ma merged commit 635c57b into release-1.9 May 3, 2023
39 checks passed
@hongzhen-ma hongzhen-ma deleted the mahz-1.9 branch May 3, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants