Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions modules/nw-ovn-kubernetes-migration.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ Perform the migration only when an interruption in service is acceptable.
. To backup the configuration for the cluster network, enter the following command:
+
----
$ oc get Network.config.openshift.io cluster -o yaml
$ oc get Network.config.openshift.io cluster -o yaml > cluster-openshift-sdn.yaml
----

. To enable the migration, set an annotation on the Cluster Network Operator configuration object by entering the following command:
Expand All @@ -43,10 +43,10 @@ $ oc patch Network.config.openshift.io cluster \
--type='merge' --patch '{ "spec": { "networkType": "OVNKubernetes" } }'
----

. To confirm the migration disabled the OpenShift SDN default CNI network provider, enter the following command. It might take several moments for all the OpenShift SDN Pods to stop.
. To confirm the migration disabled the OpenShift SDN default CNI network provider and removed all OpenShift SDN Pods, enter the following command. It might take several moments for all the OpenShift SDN Pods to stop.
+
----
$ oc get pod -n openshift-sdn --watch
$ watch oc get pod -n openshift-sdn
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dlbewley, what makes this preferable than --watch?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The condition the user wants to see is that all the pods are gone. Using --watch simply emits a stream of changes . It is much more obvious when the pods are gone with watch <cmd>, particularly if there are many pods (nodes).

If watch can be assumed to be broadly available I think the output is much more clear.

----

. To complete the migration, reboot each node in your cluster. For example, you could use a bash script similar to the following. The script assumes that you can connect to each host by using `ssh` and that you have configured `sudo` to not prompt for a password.
Expand Down
4 changes: 2 additions & 2 deletions modules/nw-ovn-kubernetes-rollback.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,10 @@ $ oc patch Network.config.openshift.io cluster \
$ oc edit Network.config.openshift.io cluster
----

. To confirm that the migration disabled the OVN-Kubernetes default CNI network provider, enter the following command. It might take several moments for all the OVN-Kubernetes Pods to stop.
. To confirm that the migration disabled the OVN-Kubernetes default CNI network provider and removed all the OVN-Kubernetes Pods, enter the following command. It might take several moments for all the OVN-Kubernetes Pods to stop.
+
----
$ oc get pod -n openshift-ovn-kubernetes --watch
$ watch oc get pod -n openshift-ovn-kubernetes
----

. To complete the rollback, reboot each node in your cluster. For example, you could use a bash script similar to the following. The script assumes that you can connect to each host by using `ssh` and that you have configured `sudo` to not prompt for a password.
Expand Down