From: Kuralamudhan Ramakrishnan <kuralamudhan.ramakrishnan@intel.com>
Date: Tue, 7 Dec 2021 19:18:11 +0000 (+0000)
Subject: Merge "Increase robustness of control plane wait"
X-Git-Url: https://gerrit.akraino.org/r/gitweb?a=commitdiff_plain;h=ec81be69009abe5d70defe38e016f14caa4a722e;hp=e0cc710b6492af36ff547bd9b3990762976b52e4;p=icn.git

Merge "Increase robustness of control plane wait"
---

diff --git a/deploy/site/vm/vm.sh b/deploy/site/vm/vm.sh
index d382420..882a8bd 100755
--- a/deploy/site/vm/vm.sh
+++ b/deploy/site/vm/vm.sh
@@ -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 {