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

ci: wait for terminating ovs-ovn pod to disappear #3160

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Aug 23, 2023

What type of this PR

Examples of user facing changes:

  • Features
  • Bug fixes
  • Docs
  • Tests

Which issue(s) this PR fixes:

Waiting for daemon set "ovs-ovn" rollout to finish: 1 out of 2 new pods have been updated...
daemon set "ovs-ovn" successfully rolled out
Collecting ovn logging files
Error from server (NotFound): pods "ovs-ovn-pt4ll" not found
Error: Process completed with exit code 1.

WHAT

🤖 Generated by Copilot at b7aa3ab

Improve the reliability of the build-x86-image workflow by using a better method to wait for ovs-ovn daemonset readiness. Change the pod counting logic in .github/workflows/build-x86-image.yaml.

🤖 Generated by Copilot at b7aa3ab

ovs-ovn waits
counting pods, not endpoints
more stable workflow

HOW

🤖 Generated by Copilot at b7aa3ab

  • Modify the condition for waiting for the ovs-ovn daemonset to be ready in four jobs in the workflow (link, link, link, link). Use the number of ovs pods and nodes instead of the ovn-nb endpoint, which may not be available in some environments.

@zhangzujian zhangzujian merged commit 8490ae6 into kubeovn:master Sep 6, 2023
59 checks passed
@zhangzujian zhangzujian deleted the fix-ci branch September 6, 2023 06:25
zhangzujian added a commit that referenced this pull request Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant