Merge pull request #46938 from shyamjvs/no-reprinting-gcloud-result

Automatic merge from submit-queue Avoid double printing output of gcloud commands in kubemark Just noticed we were unnecessarily echoing the result again. /cc @wojtek-t

Merge pull request #46938 from shyamjvs/no-reprinting-gcloud-result
c0407972 · Kubernetes Submit Queue · GitHub · 475e479c · 163f1de5 · c0407972
Commit c0407972 authored Jun 06, 2017 by Kubernetes Submit Queue Committed by GitHub Jun 06, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 1 deletion

util.sh test/kubemark/gce/util.sh +0 -1

No files found.
--- a/test/kubemark/gce/util.sh
+++ b/test/kubemark/gce/util.sh
@@ -24,7 +24,6 @@ function run-gcloud-compute-with-retries {
    # We don't use 'local' to declare gcloud_result as then ret_val always gets value 0.
    # We use tee to output to &5 (redirected to stdout) while also storing it in the variable.
    gcloud_result=$(gcloud compute "$@" 2>&1 | tee >(cat - >&5)) || local ret_val="$?"
-    echo "${gcloud_result}"
    if [[ "${ret_val:-0}" -ne "0" ]]; then
      if [[ $(echo "${gcloud_result}" | grep -c "already exists") -gt 0 ]]; then
        if [[ "${attempt}" == 1 ]]; then