From 6d2c411757197a321522687a6137a619ca192d9d Mon Sep 17 00:00:00 2001 From: CJ Cullen Date: Wed, 27 Jul 2016 10:50:28 -0700 Subject: [PATCH] Fix potential unbound KUBE_USER variable in gci/trusty. --- cluster/gce/gci/configure-helper.sh | 10 ++++++---- cluster/gce/trusty/configure-helper.sh | 10 ++++++---- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/cluster/gce/gci/configure-helper.sh b/cluster/gce/gci/configure-helper.sh index 5306d236716..2365542379e 100644 --- a/cluster/gce/gci/configure-helper.sh +++ b/cluster/gce/gci/configure-helper.sh @@ -653,10 +653,12 @@ function start-kube-apiserver { fi local -r src_dir="${KUBE_HOME}/kube-manifests/kubernetes/gci-trusty" - local -r abac_policy_json="${src_dir}/abac-authz-policy.jsonl" - remove-salt-config-comments "${abac_policy_json}" - sed -i -e "s@{{kube_user}}@${KUBE_USER}@g" "${abac_policy_json}" - cp "${abac_policy_json}" /etc/srv/kubernetes/ + if [[ -n "${KUBE_USER:-}" ]]; then + local -r abac_policy_json="${src_dir}/abac-authz-policy.jsonl" + remove-salt-config-comments "${abac_policy_json}" + sed -i -e "s@{{kube_user}}@${KUBE_USER}@g" "${abac_policy_json}" + cp "${abac_policy_json}" /etc/srv/kubernetes/ + fi src_file="${src_dir}/kube-apiserver.manifest" remove-salt-config-comments "${src_file}" diff --git a/cluster/gce/trusty/configure-helper.sh b/cluster/gce/trusty/configure-helper.sh index 2df724088d3..691cca0da3e 100644 --- a/cluster/gce/trusty/configure-helper.sh +++ b/cluster/gce/trusty/configure-helper.sh @@ -558,10 +558,12 @@ start_kube_apiserver() { src_dir="/home/kubernetes/kube-manifests/kubernetes/gci-trusty" - local -r abac_policy_json="${src_dir}/abac-authz-policy.jsonl" - remove_salt_config_comments "${abac_policy_json}" - sed -i -e "s@{{kube_user}}@${KUBE_USER}@g" "${abac_policy_json}" - cp "${abac_policy_json}" /etc/srv/kubernetes/ + if [[ -n "${KUBE_USER:-}" ]]; then + local -r abac_policy_json="${src_dir}/abac-authz-policy.jsonl" + remove_salt_config_comments "${abac_policy_json}" + sed -i -e "s@{{kube_user}}@${KUBE_USER}@g" "${abac_policy_json}" + cp "${abac_policy_json}" /etc/srv/kubernetes/ + fi src_file="${src_dir}/kube-apiserver.manifest" remove_salt_config_comments "${src_file}"