Call 'gcloud compute' only if necessary

The 'gcloud compute' is called everytime even if it is not needed. When runnin node e2e tests on RHEL, the test-e2e-node.sh script is run the way in which it takes execution path without gcloud invocation. With the current code, gcloud is called everytime. Thus introducing additional runtime dependency.

Call 'gcloud compute' only if necessary
8f28091c · Jan Chaloupka · 72fbb519 · 8f28091c
Commit 8f28091c authored Aug 28, 2016 by Jan Chaloupka
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

test-e2e-node.sh hack/make-rules/test-e2e-node.sh +2 -2

No files found.
--- a/hack/make-rules/test-e2e-node.sh
+++ b/hack/make-rules/test-e2e-node.sh
@@ -33,9 +33,9 @@ images=${IMAGES:-""}
 hosts=${HOSTS:-""}
 metadata=${INSTANCE_METADATA:-""}
 gubernator=${GUBERNATOR:-"false"}
-gci_image=$(gcloud compute images list --project google-containers \
-    --no-standard-images --regexp="gci-dev.*" --format="table[no-heading](name)")
 if [[ $hosts == "" && $images == "" ]]; then
+  gci_image=$(gcloud compute images list --project google-containers \
+    --no-standard-images --regexp="gci-dev.*" --format="table[no-heading](name)")
  images=$gci_image
  metadata="user-data<${KUBE_ROOT}/test/e2e_node/jenkins/gci-init.yaml"
 fi