From b69b94103cc9e0a3a9b4f96f86513ad2dad7b5e1 Mon Sep 17 00:00:00 2001 From: Jeff Grafton Date: Mon, 25 Apr 2016 12:10:25 -0700 Subject: [PATCH] Make detect-node-names less brittle for G{C,K}E --- cluster/gce/util.sh | 7 +++++-- cluster/gke/util.sh | 5 +---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/cluster/gce/util.sh b/cluster/gce/util.sh index d59806b5f86..345a657ec1f 100755 --- a/cluster/gce/util.sh +++ b/cluster/gce/util.sh @@ -273,13 +273,16 @@ function upload-server-tars() { function detect-node-names { detect-project INSTANCE_GROUPS=() - INSTANCE_GROUPS+=($(gcloud compute instance-groups managed list --zone "${ZONE}" --project "${PROJECT}" | grep ${NODE_INSTANCE_PREFIX} | cut -f1 -d" " || true)) + INSTANCE_GROUPS+=($(gcloud compute instance-groups managed list \ + --zone "${ZONE}" --project "${PROJECT}" \ + --regexp "${NODE_INSTANCE_PREFIX}-.+" \ + --format='value(instanceGroup)' || true)) NODE_NAMES=() if [[ -n "${INSTANCE_GROUPS[@]:-}" ]]; then for group in "${INSTANCE_GROUPS[@]}"; do NODE_NAMES+=($(gcloud compute instance-groups managed list-instances \ "${group}" --zone "${ZONE}" --project "${PROJECT}" \ - --format=yaml | grep instance: | cut -d ' ' -f 2)) + --format='value(instance)')) done echo "INSTANCE_GROUPS=${INSTANCE_GROUPS[*]}" >&2 echo "NODE_NAMES=${NODE_NAMES[*]}" >&2 diff --git a/cluster/gke/util.sh b/cluster/gke/util.sh index 54dfc410f5c..df7d8da5ee3 100755 --- a/cluster/gke/util.sh +++ b/cluster/gke/util.sh @@ -249,10 +249,7 @@ function detect-node-names { detect-node-instance-group NODE_NAMES=($(gcloud compute instance-groups managed list-instances \ "${NODE_INSTANCE_GROUP}" --zone "${ZONE}" --project "${PROJECT}" \ - --format=yaml | grep instance: | cut -d ' ' -f 2)) - - # Strip path if return value is selflink - NODE_NAMES=($(for i in ${NODE_NAMES[@]}; do basename "$i"; done)) + --format='value(instance)')) echo "NODE_NAMES=${NODE_NAMES[*]}" }