From 6224c94735adcf0d7d7c8974935a2723bbaf5ce0 Mon Sep 17 00:00:00 2001 From: Sean Eagan Date: Thu, 28 Jan 2021 16:24:08 -0600 Subject: [PATCH] Fix race condition in deploy script This fixes a race condition in the gate, where `kubectl wait` was getting called before the objects being waited on existed, which causes it to fail [0]. [0]: https://github.com/kubernetes/kubernetes/issues/83242 Change-Id: I145fa8a83bd14e673bc79e4f0d5f0cc57bb06c08 --- tools/deployment/deploy-vino.sh | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/tools/deployment/deploy-vino.sh b/tools/deployment/deploy-vino.sh index 10c2f6e..f4c8f0f 100755 --- a/tools/deployment/deploy-vino.sh +++ b/tools/deployment/deploy-vino.sh @@ -6,4 +6,13 @@ make docker-build-controller make deploy kubectl get po -A #Wait for vino controller manager Pod. +count=0 +until [[ $(kubectl -n vino-system get pod -l control-plane=controller-manager 2>/dev/null) ]]; do + count=$((count + 1)) + if [[ ${count} -eq "120" ]]; then + echo ' Timed out waiting for vino controller manager pod to exist' + return 1 + fi + sleep 2 +done kubectl wait -n vino-system pod -l control-plane=controller-manager --for=condition=ready --timeout=240s \ No newline at end of file