From 7a7cbcffd3526236129b1b07a3a6b75f2ec6c711 Mon Sep 17 00:00:00 2001 From: Jeff Grafton Date: Tue, 18 Oct 2016 17:32:56 -0700 Subject: [PATCH] Remove several red herring error messages in GCE cluster scripts --- cluster/common.sh | 10 +++++----- cluster/gce/util.sh | 3 ++- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/cluster/common.sh b/cluster/common.sh index 49844a55e4..e6be9fa524 100755 --- a/cluster/common.sh +++ b/cluster/common.sh @@ -147,15 +147,15 @@ function clear-kubeconfig() { fi local kubectl="${KUBE_ROOT}/cluster/kubectl.sh" - "${kubectl}" config unset "clusters.${CONTEXT}" - "${kubectl}" config unset "users.${CONTEXT}" - "${kubectl}" config unset "users.${CONTEXT}-basic-auth" - "${kubectl}" config unset "contexts.${CONTEXT}" - + # Unset the current-context before we delete it, as otherwise kubectl errors. local cc=$("${kubectl}" config view -o jsonpath='{.current-context}') if [[ "${cc}" == "${CONTEXT}" ]]; then "${kubectl}" config unset current-context fi + "${kubectl}" config unset "clusters.${CONTEXT}" + "${kubectl}" config unset "users.${CONTEXT}" + "${kubectl}" config unset "users.${CONTEXT}-basic-auth" + "${kubectl}" config unset "contexts.${CONTEXT}" echo "Cleared config for ${CONTEXT} from ${KUBECONFIG}" } diff --git a/cluster/gce/util.sh b/cluster/gce/util.sh index c347310bd0..64e102fab9 100755 --- a/cluster/gce/util.sh +++ b/cluster/gce/util.sh @@ -277,7 +277,7 @@ function upload-server-tars() { local staging_bucket="gs://kubernetes-staging-${project_hash}${suffix}" # Ensure the buckets are created - if ! gsutil ls "${staging_bucket}" ; then + if ! gsutil ls "${staging_bucket}" >/dev/null; then echo "Creating ${staging_bucket}" gsutil mb -l "${region}" "${staging_bucket}" fi @@ -1161,6 +1161,7 @@ function check-cluster() { # returns the result of ssh command which removes replica function remove-replica-from-etcd() { local -r port="${1}" + [[ -n "${EXISTING_MASTER_NAME}" ]] || return gcloud compute ssh "${EXISTING_MASTER_NAME}" \ --project "${PROJECT}" \ --zone "${EXISTING_MASTER_ZONE}" \