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

base: remove ovn patch for skipping ct #3140

Merged
merged 2 commits into from
Aug 17, 2023

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Aug 15, 2023

What type of this PR

Examples of user facing changes:

  • Features
  • Bug fixes
  • Docs
  • Tests

Which issue(s) this PR fixes:

Fixes #(issue-number)

WHAT

🤖 Generated by Copilot at cc73289

Increased the timeout for image building jobs in GitHub workflow and fixed a load balancer bug related to service CIDR changes. The bug fix involved moving the code that sets the load balancer service CIDR for IPv4 from init.go to syncService.

🤖 Generated by Copilot at cc73289

syncService sets CIDR
load balancer bug is fixed
autumn leaves fall slow

HOW

🤖 Generated by Copilot at cc73289

  • Remove load balancer service CIDR setting from InitOVN function to fix bug when service CIDR changes (link)
  • Increase timeout-minutes for build-x86-image and build-arm64-image jobs to avoid cancellation due to time limit (link, link)

@zhangzujian zhangzujian marked this pull request as ready for review August 15, 2023 09:18
@zhangzujian zhangzujian added the bug Something isn't working label Aug 17, 2023
@zhangzujian zhangzujian merged commit 132660e into kubeovn:release-1.11 Aug 17, 2023
46 of 51 checks passed
@zhangzujian zhangzujian deleted the fix-ct-1.11 branch August 17, 2023 09:25
@zhangzujian zhangzujian restored the fix-ct-1.11 branch August 17, 2023 09:25
@zhangzujian zhangzujian deleted the fix-ct-1.11 branch August 17, 2023 09:25
@zhangzujian zhangzujian restored the fix-ct-1.11 branch August 17, 2023 09:25
@zhangzujian zhangzujian deleted the fix-ct-1.11 branch August 17, 2023 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working network policy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants