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

Automated cherry pick of #4722: Improvement in cleanup script #4970

Conversation

luolanzone
Copy link
Contributor

Cherry pick of #4722 on release-1.11.

#4722: Improvement in cleanup script

For details on the cherry pick process, see the cherry pick requests page.

The existing script would reset ovs-vswitchd when stuck in "starting" status. In some corner cases, ovsdb-server/ovs-ovswitchd services may be removed unexpectedly.

This change includes,
- add improvement to recover ovsdb-server/ovs-vswitched service if they do not exist when running the cleanup script.
- remove vNICs using Windows VMSwitch API rather than calling ovs-vsctl commands, this can remove the dependency on the running OVS userspace process (ovs-vswitchd).

Signed-off-by: wenyingd <wenyingd@vmware.com>
@luolanzone luolanzone added the kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release label May 12, 2023
@luolanzone luolanzone requested a review from wenyingd May 12, 2023 04:09
@luolanzone
Copy link
Contributor Author

/skip-all

@luolanzone luolanzone requested a review from tnqn May 12, 2023 04:11
Copy link
Contributor

@wenyingd wenyingd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tnqn
Copy link
Member

tnqn commented May 15, 2023

/skip-all

@tnqn tnqn merged commit 944c9de into antrea-io:release-1.11 May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants