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

e2e: fix switch lb rule test #3071

Merged
merged 1 commit into from
Jul 25, 2023
Merged

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Jul 25, 2023

What type of this PR

  • Tests

Which issue(s) this PR fixes:

Fixes occasional address conflict in switch lb rule e2e test.

WHAT

🤖 Generated by Copilot at d5a21ad

Remove pool annotation from statefulset test case. Use kube-ovn IPAM feature instead.

🤖 Generated by Copilot at d5a21ad

No more pool annotation
Statefulset pods use IPAM
Simpler and better

HOW

🤖 Generated by Copilot at d5a21ad

  • Remove the pool annotation and the code that assigns random IPs to the statefulset pods (link). This simplifies the test case and aligns with the best practices of using kube-ovn IPAM feature to allocate IPs from the subnet. The file switch_lb_rule.go contains the test case for verifying the switch load balancing rule functionality.

@zhangzujian zhangzujian added the test automation tests label Jul 25, 2023
@zhangzujian zhangzujian requested a review from bobz965 July 25, 2023 05:07
@zhangzujian zhangzujian merged commit 038ff7d into kubeovn:master Jul 25, 2023
59 checks passed
@zhangzujian zhangzujian deleted the fix-e2e branch July 25, 2023 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test automation tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants