From de25cfb60d077ea7a57b07b64bcec7f2f09003c1 Mon Sep 17 00:00:00 2001
From: Myers Carpenter <myers@maski.org>
Date: Tue, 8 Jan 2019 15:41:10 -0500
Subject: [PATCH] check for required settings

---
 update.sh | 74 ++++++++++++++++++++++++++++++++++---------------------
 1 file changed, 46 insertions(+), 28 deletions(-)

diff --git a/update.sh b/update.sh
index 479aa14..1b1c729 100755
--- a/update.sh
+++ b/update.sh
@@ -1,52 +1,70 @@
 #!/bin/bash
 
-set -e
+set -euo pipefail
 
 export KUBECONFIG=/tmp/kubeconfig
 
-if [ -z ${PLUGIN_NAMESPACE} ]; then
+if [ -z "${PLUGIN_KUBERNETES_TOKEN:-}" ]; then
+  echo "ERROR: You must set 'kubernetes_token'"
+  exit 1
+fi
+
+if [ -z "${PLUGIN_TAG:-}" ]; then
+  echo "ERROR: You must set 'tag'"
+  exit 1
+fi
+
+if [ -z "${PLUGIN_REPO:-}" ]; then
+  echo "ERROR: You must set 'repo'"
+  exit 1
+fi
+
+if [ -z "${PLUGIN_KUBERNETES_SERVER:-}" ]; then
+  echo "ERROR: You must set 'kubernetes_server'"
+  exit 1
+fi
+
+if [ -z "${PLUGIN_DEPLOYMENT:-}" ]; then
+  echo "ERROR: You must set 'deployment'"
+  exit 1
+fi
+
+if [ -z "${PLUGIN_CONTAINER:-}" ]; then
+  echo "ERROR: You must set 'container'"
+  exit 1
+fi
+
+if [ -z "${PLUGIN_NAMESPACE:-}" ]; then
   PLUGIN_NAMESPACE="default"
 fi
 
-if [ -z ${PLUGIN_KUBERNETES_USER} ]; then
+if [ -z "${PLUGIN_KUBERNETES_USER:-}" ]; then
   PLUGIN_KUBERNETES_USER="default"
 fi
 
-if [ ! -z ${PLUGIN_KUBERNETES_TOKEN} ]; then
-  KUBERNETES_TOKEN=$PLUGIN_KUBERNETES_TOKEN
-fi
-
-if [ ! -z ${PLUGIN_KUBERNETES_SERVER} ]; then
-  KUBERNETES_SERVER=$PLUGIN_KUBERNETES_SERVER
-fi
-
-if [ ! -z ${PLUGIN_KUBERNETES_CERT} ]; then
-  KUBERNETES_CERT=${PLUGIN_KUBERNETES_CERT}
-fi
-
-kubectl config set-credentials default --token=${KUBERNETES_TOKEN}
-if [ ! -z ${KUBERNETES_CERT} ]; then
-  echo ${KUBERNETES_CERT} | base64 -d > ca.crt
-  kubectl config set-cluster default --server=${KUBERNETES_SERVER} --certificate-authority=ca.crt
-else
+kubectl config set-credentials default --token=${PLUGIN_KUBERNETES_TOKEN}
+if [ -z "${PLUGIN_KUBERNETES_CERT:-}" ]; then
   echo "WARNING: Using insecure connection to cluster"
-  kubectl config set-cluster default --server=${KUBERNETES_SERVER} --insecure-skip-tls-verify=true
+  kubectl config set-cluster default --server=${PLUGIN_KUBERNETES_SERVER} --insecure-skip-tls-verify=true
+else
+  echo "${PLUGIN_KUBERNETES_CERT}" | base64 -d > ca.crt
+  kubectl config set-cluster default --server=${PLUGIN_KUBERNETES_SERVER} --certificate-authority=ca.crt
 fi
 
 kubectl config set-context default --cluster=default --user=${PLUGIN_KUBERNETES_USER}
 kubectl config use-context default
 
-# kubectl version
-IFS=',' read -r -a DEPLOYMENTS <<< "${PLUGIN_DEPLOYMENT}"
-IFS=',' read -r -a CONTAINERS <<< "${PLUGIN_CONTAINER}"
+IFS=',' read -r -a DEPLOYMENTS <<< "${PLUGIN_DEPLOYMENT:-}"
+IFS=',' read -r -a CONTAINERS <<< "${PLUGIN_CONTAINER:-}"
 for DEPLOY in ${DEPLOYMENTS[@]}; do
-  echo Deploying to $KUBERNETES_SERVER
+  echo Deploying to $PLUGIN_KUBERNETES_SERVER
   for CONTAINER in ${CONTAINERS[@]}; do
-    if [[ ${PLUGIN_FORCE} == "true" ]]; then
+    if [ ${PLUGIN_FORCE:-} == "true" ]; then
+      # force changing the image by changing it to the TAG + FORCE then changing it back
       kubectl -n ${PLUGIN_NAMESPACE} set image deployment/${DEPLOY} \
-        ${CONTAINER}=${PLUGIN_REPO}:${PLUGIN_TAG}FORCE
+        ${CONTAINER}=${PLUGIN_REPO}:${PLUGIN_TAG:-}FORCE
     fi
     kubectl -n ${PLUGIN_NAMESPACE} set image deployment/${DEPLOY} \
-      ${CONTAINER}=${PLUGIN_REPO}:${PLUGIN_TAG} --record
+      ${CONTAINER}=${PLUGIN_REPO}:${PLUGIN_TAG:-} --record
   done
 done