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.

Sign up for GitHub

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

Jump to bottom

fix repeate set chassis #3083

Merged
merged 4 commits into from
Jul 28, 2023
Merged

fix repeate set chassis #3083

merged 4 commits into from
Jul 28, 2023

Conversation

zbb88888
Copy link
Collaborator

@zbb88888 zbb88888 commented Jul 27, 2023 •
edited
Loading

What type of this PR

Examples of user facing changes:

  • Bug fixes

Which issue(s) this PR fixes:

Fixes #3084

WHAT

🤖 Generated by Copilot at bad86a3

Improve OVN controller performance by skipping chassis updates for nodes with matching ids. Add a chassis id check in pkg/controller/init.go.

🤖 Generated by Copilot at bad86a3

chassis id check
skips nodes with matching names
snowflakes in OVN

HOW

🤖 Generated by Copilot at bad86a3

  • Add a check to skip nodes with matching chassis id and name (link) to avoid unnecessary updates to the OVN database

@zbb88888 zbb88888 marked this pull request as ready for review July 27, 2023 13:23
@zbb88888 zbb88888 requested a review from zhangzujian July 28, 2023 01:13
@zbb88888
Copy link
Collaborator Author

hi @zhangzujian , could you help review this ?

@zbb88888 zbb88888 merged commit fa91428 into kubeovn:master Jul 28, 2023
@zbb88888 zbb88888 deleted the fix-repeat-chassis-set branch July 28, 2023 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhangzujian zhangzujian zhangzujian approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix repeat set sb db
2 participants
@zbb88888 @zhangzujian