Increase robustness of control plane wait 46/4546/1
authorTodd Malsbary <todd.malsbary@intel.com>
Thu, 2 Dec 2021 18:30:16 +0000 (10:30 -0800)
committerTodd Malsbary <todd.malsbary@intel.com>
Tue, 7 Dec 2021 17:54:21 +0000 (09:54 -0800)
Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
Change-Id: Ifa5c051aa2435f27b3dc9a8d934b510118ed070c

deploy/site/vm/vm.sh

index 0d79743..8dcc359 100755 (executable)
@@ -41,7 +41,11 @@ function is_cluster_ready {
 }
 
 function is_control_plane_ready {
-    [[ $(kubectl --kubeconfig=${BUILDDIR}/e2etest-admin.conf get nodes -l node-role.kubernetes.io/control-plane -o jsonpath='{range .items[*]}{.status.conditions[?(@.type=="Ready")].status}{"\n"}{end}' | grep -c -v True) == 0 ]]
+    # Checking the Cluster resource status is not sufficient, it
+    # reports the control plane as ready before the nodes forming the
+    # control plane are ready
+    local -r replicas=$(kubectl -n metal3 get kubeadmcontrolplane e2etest -o jsonpath='{.spec.replicas}')
+    [[ $(kubectl --kubeconfig=${BUILDDIR}/e2etest-admin.conf get nodes -l node-role.kubernetes.io/control-plane -o jsonpath='{range .items[*]}{.status.conditions[?(@.type=="Ready")].status}{"\n"}{end}' | grep -c True) == ${replicas} ]]
 }
 
 function wait_for_all_ready {