From 89501a88656b04d1aa362101480d485a2a42caeb Mon Sep 17 00:00:00 2001
From: Madhu Rajanna <madhupr007@gmail.com>
Date: Thu, 15 Feb 2024 13:32:51 +0100
Subject: [PATCH] ci: use kubectl_retry helper for kubectl

use kubectl_retry helper to avoid
intermediate failure and retry again

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
---
 scripts/install-snapshot.sh | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

diff --git a/scripts/install-snapshot.sh b/scripts/install-snapshot.sh
index 91e8fa780..20b57596f 100755
--- a/scripts/install-snapshot.sh
+++ b/scripts/install-snapshot.sh
@@ -7,6 +7,9 @@ SCRIPT_DIR="$(dirname "${0}")"
 # shellcheck source=build.env
 source "${SCRIPT_DIR}/../build.env"
 
+# shellcheck disable=SC1091
+[ ! -e "${SCRIPT_DIR}"/utils.sh ] || source "${SCRIPT_DIR}"/utils.sh
+
 SNAPSHOT_VERSION=${SNAPSHOT_VERSION:-"v5.0.1"}
 
 TEMP_DIR="$(mktemp -d)"
@@ -34,19 +37,19 @@ function install_snapshot_controller() {
 
     create_or_delete_resource "create" "${namespace}"
 
-    pod_ready=$(kubectl get pods -l app=snapshot-controller -n "${namespace}" -o jsonpath='{.items[0].status.containerStatuses[0].ready}')
+    pod_ready=$(kubectl_retry get pods -l app=snapshot-controller -n "${namespace}" -o jsonpath='{.items[0].status.containerStatuses[0].ready}')
     INC=0
     until [[ "${pod_ready}" == "true" || $INC -gt 20 ]]; do
         sleep 10
         ((++INC))
-        pod_ready=$(kubectl get pods -l app=snapshot-controller -n "${namespace}" -o jsonpath='{.items[0].status.containerStatuses[0].ready}')
+        pod_ready=$(kubectl_retry get pods -l app=snapshot-controller -n "${namespace}" -o jsonpath='{.items[0].status.containerStatuses[0].ready}')
         echo "snapshotter pod status: ${pod_ready}"
     done
 
     if [ "${pod_ready}" != "true" ]; then
         echo "snapshotter controller creation failed"
-        kubectl get pods -l app=snapshot-controller -n "${namespace}"
-        kubectl describe po -l app=snapshot-controller -n "${namespace}"
+        kubectl_retry get pods -l app=snapshot-controller -n "${namespace}"
+        kubectl_retry describe po -l app=snapshot-controller -n "${namespace}"
         exit 1
     fi
 
@@ -88,14 +91,14 @@ function create_or_delete_resource() {
         fi
     fi
 
-    kubectl "${operation}" -f "${VOLUME_GROUP_SNAPSHOTCLASS}"
-    kubectl "${operation}" -f "${VOLUME_GROUP_SNAPSHOT_CONTENT}"
-    kubectl "${operation}" -f "${VOLUME_GROUP_SNAPSHOT}"
-    kubectl "${operation}" -f "${temp_rbac}"
-    kubectl "${operation}" -f "${temp_snap_controller}" -n "${namespace}"
-    kubectl "${operation}" -f "${SNAPSHOTCLASS}"
-    kubectl "${operation}" -f "${VOLUME_SNAPSHOT_CONTENT}"
-    kubectl "${operation}" -f "${VOLUME_SNAPSHOT}"
+    kubectl_retry "${operation}" -f "${VOLUME_GROUP_SNAPSHOTCLASS}"
+    kubectl_retry "${operation}" -f "${VOLUME_GROUP_SNAPSHOT_CONTENT}"
+    kubectl_retry "${operation}" -f "${VOLUME_GROUP_SNAPSHOT}"
+    kubectl_retry "${operation}" -f "${temp_rbac}"
+    kubectl_retry "${operation}" -f "${temp_snap_controller}" -n "${namespace}"
+    kubectl_retry "${operation}" -f "${SNAPSHOTCLASS}"
+    kubectl_retry "${operation}" -f "${VOLUME_SNAPSHOT_CONTENT}"
+    kubectl_retry "${operation}" -f "${VOLUME_SNAPSHOT}"
 }
 
 case "${1:-}" in