commit 46c17796c84b41d91023af310c49fabe47083363
author Frederic Gurr <frederic.gurr@eclipse-foundation.org> Mon Jun 13 12:31:09 2022 +0200
committer Frederic Gurr <frederic.gurr@eclipse-foundation.org> Mon Jun 13 12:31:09 2022 +0200
tree 4c6a9371f778ad502f546fe55e6e5ba4224d6cbb
parent fcde85f14336bb190f6a3801412f918839772055

Add quotes around variables

Change-Id: Ic68b36004ed726b62f8733cf7c94ab620baa0a2e

infoCenter/k8s/createInfoCenter_yaml.sh

diff --git a/infoCenter/k8s/createInfoCenter_yaml.sh b/infoCenter/k8s/createInfoCenter_yaml.sh
index 4c38587..ffc09ce 100755
--- a/infoCenter/k8s/createInfoCenter_yaml.sh
+++ b/infoCenter/k8s/createInfoCenter_yaml.sh
@@ -34,7 +34,7 @@
 create_license_header() {
   local file="${1:-}"
   local year="2021"
-  cat <<EOF > ${file}
+  cat <<EOF > "${file}"
 #*******************************************************************************
 # Copyright (c) ${year} Eclipse Foundation and others.
 # This program and the accompanying materials are made available
@@ -52,7 +52,7 @@
   local host_name="${3:-}"
   local file_name="${release_name}/route.yml"
   create_license_header "${file_name}"
-  cat <<EOF >> ${file_name}
+  cat <<EOF >> "${file_name}"
 apiVersion: "route.openshift.io/v1"
 kind: "Route"
 metadata:
@@ -83,7 +83,7 @@
   local namespace_name="${2:-}"
   local file_name="${release_name}/service.yml"
   create_license_header "${file_name}"
-  cat <<EOF >> ${file_name}
+  cat <<EOF >> "${file_name}"
 apiVersion: "v1"
 kind: "Service"
 metadata:
@@ -113,7 +113,7 @@
   echo "Image name: ${infocenter_image}"
   
   create_license_header "${file_name}"
-  cat <<EOF >> ${file_name}
+  cat <<EOF >> "${file_name}"
 apiVersion: apps/v1
 kind: Deployment
 metadata:

infoCenter/k8s/deployInfoCenter.sh

diff --git a/infoCenter/k8s/deployInfoCenter.sh b/infoCenter/k8s/deployInfoCenter.sh
index 2a6c5b9..cb20754 100755
--- a/infoCenter/k8s/deployInfoCenter.sh
+++ b/infoCenter/k8s/deployInfoCenter.sh
@@ -15,7 +15,7 @@
 set -o pipefail
 
 # Parameters:
-release_name=${1:-}
+release_name="${1:-}"
 
 # Verify inputs
 if [[ -z "${release_name}" ]]; then

infoCenter/k8s/removeInfoCenter.sh

diff --git a/infoCenter/k8s/removeInfoCenter.sh b/infoCenter/k8s/removeInfoCenter.sh
index d2b8399..f0ca7b0 100755
--- a/infoCenter/k8s/removeInfoCenter.sh
+++ b/infoCenter/k8s/removeInfoCenter.sh
@@ -15,7 +15,7 @@
 set -o pipefail
 
 # Parameters:
-release_name=${1:-}
+release_name="${1:-}"
 namespace="infocenter"
 
 # Verify inputs
@@ -24,15 +24,15 @@
   exit 1
 fi
 
-oc scale deployment infocenter-${release_name} -n="${namespace}" --replicas=0
-oc delete deployment infocenter-${release_name} -n "${namespace}"
-oc delete service infocenter-${release_name} -n "${namespace}"
-oc delete route infocenter-${release_name} -n "${namespace}"
+oc scale deployment "infocenter-${release_name}" -n="${namespace}" --replicas=0
+oc delete deployment "infocenter-${release_name}" -n "${namespace}"
+oc delete service "infocenter-${release_name}" -n "${namespace}"
+oc delete route "infocenter-${release_name}" -n "${namespace}"
 
 remove_question() {
   read -p "Do you want to remove the folder for ${release_name}? (Y)es, (N)o, E(x)it: " yn
   case $yn in
-    [Yy]* ) rm -rf ${release_name};;
+    [Yy]* ) rm -rf "${release_name}";;
     [Nn]* ) exit 0;;
     [Xx]* ) exit 0;;
         * ) echo "Please answer (Y)es, (N)o, E(x)it"; question;