Merge pull request #75915 from xichengliudui/fixshellcheckfailures-05

fix shellcheck failures in ./hack/jenkins/...

hack/.shellcheck_failures

@@ -27,11 +27,8 @@
 ./hack/cherry_pick_pull.sh
 ./hack/ginkgo-e2e.sh
 ./hack/grab-profiles.sh
-./hack/jenkins/benchmark-dockerized.sh
-./hack/jenkins/build.sh
 ./hack/jenkins/test-dockerized.sh
 ./hack/jenkins/upload-to-gcs.sh
-./hack/jenkins/verify-dockerized.sh
 ./hack/lib/init.sh
 ./hack/lib/logging.sh
 ./hack/lib/protoc.sh

hack/jenkins/benchmark-dockerized.sh

@@ -21,7 +21,12 @@ set -o xtrace
 
 retry() {
   for i in {1..5}; do
-    "$@" && return 0 || sleep "${i}"
+    if "$@"
+    then      
+      return 0
+    else
+      sleep "${i}"
+    fi
   done
   "$@"
 }
@@ -51,5 +56,5 @@ cd /go/src/k8s.io/kubernetes
 # Run the benchmark tests and pretty-print the results into a separate file.
 make test-integration WHAT="$*" KUBE_TEST_ARGS="-run='XXX' -bench=. -benchmem" \
   | tee \
-   >(prettybench -no-passthrough > ${ARTIFACTS}/BenchmarkResults.txt) \
-   >(go run test/integration/benchmark/jsonify/main.go ${ARTIFACTS}/BenchmarkResults_benchmark_$(date -u +%Y-%m-%dT%H:%M:%SZ).json || cat > /dev/null)
+   >(prettybench -no-passthrough > "${ARTIFACTS}/BenchmarkResults.txt") \
+   >(go run test/integration/benchmark/jsonify/main.go "${ARTIFACTS}"/BenchmarkResults_benchmark_"$(date -u +%Y-%m-%dT%H:%M:%SZ)".json || cat > /dev/null)

hack/jenkins/build.sh

@@ -35,7 +35,7 @@ export PATH=${PATH}:/usr/local/go/bin
 # Skip gcloud update checking
 export CLOUDSDK_COMPONENT_MANAGER_DISABLE_UPDATE_CHECK=true
 
-: ${KUBE_RELEASE_RUN_TESTS:="n"}
+: "${KUBE_RELEASE_RUN_TESTS:="n"}"
 export KUBE_RELEASE_RUN_TESTS
 
 # Clean stuff out. Assume the last build left the tree in an odd
@@ -58,8 +58,8 @@ if [[ ${KUBE_SKIP_PUSH_GCS:-} =~ ^[yY]$ ]]; then
   echo "Not pushed to GCS..."
 else
   readonly release_infra_clone="${WORKSPACE}/_tmp/release.git"
-  mkdir -p ${WORKSPACE}/_tmp
-  git clone https://github.com/kubernetes/release ${release_infra_clone}
+  mkdir -p "${WORKSPACE}/_tmp"
+  git clone https://github.com/kubernetes/release "${release_infra_clone}"
 
   push_build=${release_infra_clone}/push-build.sh
 
@@ -67,7 +67,7 @@ else
     && bucket_flag="--bucket=${KUBE_GCS_RELEASE_BUCKET-}"
   [[ -n "${KUBE_GCS_RELEASE_SUFFIX-}" ]] \
     && gcs_suffix_flag="--gcs-suffix=${KUBE_GCS_RELEASE_SUFFIX-}"
-  ${push_build} ${bucket_flag-} ${gcs_suffix_flag-} \
+  "${push_build}" "${bucket_flag-}" "${gcs_suffix_flag-}" \
     --nomock --verbose --ci
 fi
 

hack/jenkins/verify-dockerized.sh

@@ -21,7 +21,12 @@ set -o xtrace
 
 retry() {
   for i in {1..5}; do
-    "$@" && return 0 || sleep "${i}"
+    if "$@"
+    then
+      return 0
+    else
+      sleep "${i}"
+    fi
   done
   "$@"
 }