X-Git-Url: https://gerrit.akraino.org/r/gitweb?a=blobdiff_plain;f=deploy%2Fsite%2Fvm%2Fvm.sh;h=882a8bda26532bee7c5fcd6da6b1ed3e63ebdf15;hb=c910d76976e8c2698a7895a2f95f46f55fc0c73d;hp=1fd980ae0782ef84b1050d0a4d224ec8a82285e1;hpb=26141fccdb4824f330e9fa1607f15f2a0048fb82;p=icn.git

diff --git a/deploy/site/vm/vm.sh b/deploy/site/vm/vm.sh
index 1fd980a..882a8bd 100755
--- a/deploy/site/vm/vm.sh
+++ b/deploy/site/vm/vm.sh
@@ -23,7 +23,7 @@ function release_name {
 }
 
 function deploy {
-    for values in build/site/vm/machine-*-values.yaml; do
+    for values in ${BUILDDIR}/machine-*-values.yaml; do
 	helm -n metal3 install $(release_name ${values}) ${SCRIPTDIR}/../../machine --create-namespace -f ${values}
     done
     helm -n metal3 install cluster-e2etest ${SCRIPTDIR}/../../cluster --create-namespace -f ${BUILDDIR}/cluster-e2etest-values.yaml
@@ -31,7 +31,7 @@ function deploy {
 
 function clean {
     helm -n metal3 uninstall cluster-e2etest
-    for values in build/site/vm/machine-*-values.yaml; do
+    for values in ${BUILDDIR}/machine-*-values.yaml; do
 	helm -n metal3 uninstall $(release_name ${values})
     done
 }
@@ -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 {