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

ovs: change update strategy to RollingUpdate #2422

Merged
merged 5 commits into from
Mar 7, 2023

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Mar 3, 2023

What type of this PR

  • Enhancement

How this PR works

When ovs-ovn exits, the shell script checks whether there is a newer daemonset revision(compared to itself). If a newer
revision is found, the ovsdb-server/ovs-vswitchd/ovn-controller created by this pod will not be stopped. Instead, these processes will be stopped later by the pod with a newer revision.

NOTICE
This patch requires the feature gate DaemonSetUpdateSurge.

Feature Default Stage Since Until
DaemonSetUpdateSurge false Alpha 1.21 1.21
DaemonSetUpdateSurge true Beta 1.22 1.24
DaemonSetUpdateSurge true GA 1.25 -
$ kubectl -n kube-system get po -l app=ovs -o wide -w
NAME            READY   STATUS              RESTARTS   AGE   IP           NODE
ovs-ovn-crk7c   1/1     Running             0          40s   172.18.0.3   kube-ovn-control-plane
ovs-ovn-f4q62   1/1     Running             0          40s   172.18.0.2   kube-ovn-worker
ovs-ovn-gtn6s   0/1     Pending             0          0s    <none>       <none>
ovs-ovn-gtn6s   0/1     Pending             0          0s    <none>       kube-ovn-worker
ovs-ovn-nk9hc   0/1     Pending             0          0s    <none>       <none>
ovs-ovn-nk9hc   0/1     Pending             0          0s    <none>       kube-ovn-control-plane
ovs-ovn-gtn6s   0/1     ContainerCreating   0          0s    172.18.0.2   kube-ovn-worker
ovs-ovn-nk9hc   0/1     ContainerCreating   0          0s    172.18.0.3   kube-ovn-control-plane
ovs-ovn-nk9hc   0/1     Running             0          1s    172.18.0.3   kube-ovn-control-plane
ovs-ovn-gtn6s   0/1     Running             0          2s    172.18.0.2   kube-ovn-worker
ovs-ovn-gtn6s   1/1     Running             0          11s   172.18.0.2   kube-ovn-worker
ovs-ovn-f4q62   1/1     Terminating         0          54s   172.18.0.2   kube-ovn-worker
ovs-ovn-nk9hc   1/1     Running             0          11s   172.18.0.3   kube-ovn-control-plane
ovs-ovn-crk7c   1/1     Terminating         0          54s   172.18.0.3   kube-ovn-control-plane
ovs-ovn-f4q62   0/1     Terminating         0          61s   172.18.0.2   kube-ovn-worker
ovs-ovn-f4q62   0/1     Terminating         0          61s   172.18.0.2   kube-ovn-worker
ovs-ovn-f4q62   0/1     Terminating         0          61s   172.18.0.2   kube-ovn-worker
ovs-ovn-crk7c   0/1     Terminating         0          61s   172.18.0.3   kube-ovn-control-plane
ovs-ovn-crk7c   0/1     Terminating         0          61s   172.18.0.3   kube-ovn-control-plane
ovs-ovn-crk7c   0/1     Terminating         0          61s   172.18.0.3   kube-ovn-control-plane

@zhangzujian zhangzujian linked an issue Mar 3, 2023 that may be closed by this pull request
@zhangzujian zhangzujian marked this pull request as ready for review March 3, 2023 06:23
@zhangzujian zhangzujian requested a review from oilbeater March 3, 2023 06:23
@zhangzujian zhangzujian added the enhancement Improve exist functions label Mar 3, 2023
@oilbeater
Copy link
Collaborator

Need to consider how we can upgrade from previous versions

@zhangzujian zhangzujian merged commit 497260e into kubeovn:master Mar 7, 2023
@zhangzujian zhangzujian deleted the ovs-rollout branch March 7, 2023 09:00
@zhangzujian zhangzujian mentioned this pull request Mar 11, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci enhancement Improve exist functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decrease down time when restart ovs-ovn
2 participants