More defaults

Author: simonklb

scripts/migration/helmfile.sh

@@ -41,7 +41,7 @@ helmfile_apply() {
   prefix="${1}"
   shift
 
-  if [[ "${CK8S_DRY_RUN_INSTALL}" == "true" ]]; then
+  if [[ "${CK8S_DRY_RUN_INSTALL:-"false"}" == "true" ]]; then
     log_info "Dry-running on ${*}"
     helmfile_diff "${prefix}" "${@}"
     return
@@ -70,7 +70,7 @@ helmfile_destroy() {
   prefix="${1}"
   shift
 
-  if [[ "${CK8S_DRY_RUN_INSTALL}" == "true" ]]; then
+  if [[ "${CK8S_DRY_RUN_INSTALL:-"false"}" == "true" ]]; then
     log_info "Dry-run: Destroy on ${*} ${prefix}"
     return
   fi
@@ -152,7 +152,7 @@ helmfile_upgrade() {
     if ! helmfile_apply "${1}" "namespace=${2},name=${3}"; then
       log_error "error: failed to upgrade ${1} ${2}/${3}"
 
-      if [[ "${CK8S_DRY_RUN_INSTALL}" == "true" ]]; then
+      if [[ "${CK8S_DRY_RUN_INSTALL:-"false"}" == "true" ]]; then
         return 1
       fi
 

scripts/migration/kubectl.sh

@@ -11,7 +11,7 @@ kubectl_do() {
     log_fatal "Second argument must be a kubectl operation, not an option"
   fi
 
-  if [[ "${CK8S_DRY_RUN_INSTALL}" == "true" ]]; then
+  if [[ "${CK8S_DRY_RUN_INSTALL:-"false"}" == "true" ]]; then
     local command="${2}"
     # List of common commands that generally DO NOT support --dry-run=server
     case "${command}" in