Update "Migrate to Cilium" how-to

appuio · May 16, 2024 · 3a663d0 · 3a663d0
1 parent 4258bd5
commit 3a663d0
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/docs/modules/ROOT/pages/how-tos/network/migrate-to-cilium.adoc b/docs/modules/ROOT/pages/how-tos/network/migrate-to-cilium.adoc
@@ -174,7 +174,7 @@ HOST_PREFIX=$(kubectl get network.config cluster \
 
 yq -i '.parameters.cilium.cilium_helm_values.ipam.operator.clusterPoolIPv4MaskSize = "'"${HOST_PREFIX}"'"' \
   "${CLUSTER_ID}.yml"
-yq -i '.parameters.cilium.cilium_helm_values.ipam.operator.clusterPoolIPv4PodCIDR = "'"${POD_CIDR}"'"' \
+yq -i '.parameters.cilium.cilium_helm_values.ipam.operator.~clusterPoolIPv4PodCIDRList = "'"${POD_CIDR}"'"' \
   "${CLUSTER_ID}.yml"
 ----
 
@@ -196,6 +196,11 @@ commodore catalog compile "${CLUSTER_ID}"
 
 . Patch cluster network config
 +
+[IMPORTANT]
+====
+Only execute this step after you've paused all machine config pools.
+====
++
 [source,bash]
 ----
 kubectl --as=cluster-admin patch network.config cluster \
@@ -259,6 +264,16 @@ You may need to grab the cluster-admin credentials to complete this step since t
 +
 [TIP]
 ====
+You may want to restart the multus daemonset once the old CNI pods are removed.
+
+[source,bash]
+----
+kubectl --as=cluster-admin -n openshift-multus rollout restart ds/multus
+----
+====
++
+[TIP]
+====
 It may be necessary to force drain nodes manually to allow the machine-config-operator to reboot the nodes.
 Use `kubectl --as=cluster-admin drain --ignore-daemonsets --delete-emptydir-data --force --disable-eviction` to circumvent PDB violations if necessary.