Fix some linter warnings, part 2

kafka-init/wait-for.sh

@@ -2,7 +2,7 @@
 # Use this script to test if a given TCP host/port are available
 # https://github.com/vishnubob/wait-for-it/blob/master/wait-for-it.sh
 
-cmdname=$(basename $0)
+cmdname=$(basename "$0")
 
 echoerr() { if [[ $QUIET -ne 1 ]]; then echo "$@" 1>&2; fi }
 
@@ -34,10 +34,10 @@ wait_for()
     while :
     do
         if [[ $ISBUSY -eq 1 ]]; then
-            nc -z $HOST $PORT
+            nc -z "$HOST" "$PORT"
             result=$?
         else
-            (echo > /dev/tcp/$HOST/$PORT) >/dev/null 2>&1
+            (echo > /dev/tcp/"$HOST"/"$PORT") >/dev/null 2>&1
             result=$?
         fi
         if [[ $result -eq 0 ]]; then
@@ -54,9 +54,9 @@ wait_for_wrapper()
 {
     # In order to support SIGINT during timeout: http://unix.stackexchange.com/a/57692
     if [[ $QUIET -eq 1 ]]; then
-        timeout $BUSYTIMEFLAG $TIMEOUT $0 --quiet --child --host=$HOST --port=$PORT --timeout=$TIMEOUT &
+        timeout "$BUSYTIMEFLAG" "$TIMEOUT" $0 --quiet" --child --host="$HOST" --port="$PORT" --timeout="$TIMEOUT" &
     else
-        timeout $BUSYTIMEFLAG $TIMEOUT $0 --child --host=$HOST --port=$PORT --timeout=$TIMEOUT &
+        timeout "$BUSYTIMEFLAG" "$TIMEOUT" $0 --child --host="$HOST" --port="$PORT" --timeout="$TIMEOUT" &
     fi
     PID=$!
     trap "kill -INT -$PID" INT
@@ -172,7 +172,7 @@ if [[ $CLI != "" ]]; then
         echoerr "$cmdname: strict mode, refusing to execute subprocess"
         exit $RESULT
     fi
-    exec $CLI
+    exec "$CLI"
 else
-    exit $RESULT
+    exit "$RESULT"
 fi

kafka/start.sh

@@ -22,21 +22,21 @@ export STATE_CHANGE_LOG_LEVEL=${STATE_CHANGE_LOG_LEVEL:-"INFO"}
 export AUTHORIZER_LOG_LEVEL=${AUTHORIZER_LOG_LEVEL:-"WARN"}
 GC_LOG_ENABLED=${GC_LOG_ENABLED:-"False"}
 
-first_zk=$(echo $ZOOKEEPER_CONNECTION_STRING | cut -d, -f1)
-zk_host=$(echo $first_zk | cut -d\: -f1)
-zk_port=$(echo $first_zk | cut -d\: -f2)
+first_zk=$(echo "$ZOOKEEPER_CONNECTION_STRING" | cut -d, -f1)
+zk_host=$(echo "$first_zk" | cut -d\: -f1)
+zk_port=$(echo "$first_zk" | cut -d\: -f2)
 
 # wait for zookeeper to become available
 if [ "$ZOOKEEPER_WAIT" = "true" ]; then
   success="false"
-  for i in $(seq $ZOOKEEPER_WAIT_RETRIES); do
-    ok=$(echo ruok | nc $zk_host $zk_port -w $ZOOKEEPER_WAIT_TIMEOUT)
+  for i in $(seq "$ZOOKEEPER_WAIT_RETRIES"); do
+    ok=$(echo ruok | nc "$zk_host" "$zk_port" -w "$ZOOKEEPER_WAIT_TIMEOUT")
     if [ $? -eq 0 -a "$ok" = "imok" ]; then
       success="true"
       break
     else
       echo "Connect attempt $i of $ZOOKEEPER_WAIT_RETRIES failed, retrying..."
-      sleep $ZOOKEEPER_WAIT_DELAY
+      sleep "$ZOOKEEPER_WAIT_DELAY"
     fi
   done
 
@@ -69,7 +69,7 @@ for f in $CONFIG_TEMPLATES/*.properties.j2; do
 done
 
 if [ -z "$KAFKA_HEAP_OPTS" ]; then
-  max_heap=$(python /heap.py $KAFKA_MAX_HEAP_MB)
+  max_heap=$(python /heap.py "$KAFKA_MAX_HEAP_MB")
   KAFKA_HEAP_OPTS="-Xmx${max_heap} -Xms${max_heap}"
   export KAFKA_HEAP_OPTS
 fi

keystone/keystone-bootstrap.sh

@@ -41,15 +41,15 @@ fi
 if [[ -e /db-init ]]; then
     echo "Creating bootstrap credentials..."
     keystone-manage bootstrap \
-        --bootstrap-password $admin_password \
-        --bootstrap-username $admin_username \
-        --bootstrap-project-name $admin_project \
-        --bootstrap-role-name $admin_role \
-        --bootstrap-service-name $admin_service \
-        --bootstrap-region-id $admin_region \
-        --bootstrap-admin-url $admin_url \
-        --bootstrap-public-url $public_url \
-        --bootstrap-internal-url $internal_url
+        --bootstrap-password "$admin_password" \
+        --bootstrap-username "$admin_username" \
+        --bootstrap-project-name "$admin_project" \
+        --bootstrap-role-name "$admin_role" \
+        --bootstrap-service-name "$admin_service" \
+        --bootstrap-region-id "$admin_region" \
+        --bootstrap-admin-url "$admin_url" \
+        --bootstrap-public-url "$public_url" \
+        --bootstrap-internal-url "$internal_url"
 
     sleep 5
 

keystone/start.sh

@@ -11,7 +11,7 @@ if [[ "$KEYSTONE_DATABASE_BACKEND" =  "mysql" ]]; then
 
     echo "Waiting for mysql to become available..."
     success="false"
-    for i in $(seq $retries); do
+    for i in $(seq "$retries"); do
         mysqladmin status \
             --host="$mysql_host" \
             --user="$mysql_user" \
@@ -40,7 +40,7 @@ if [[ "$KEYSTONE_DATABASE_BACKEND" =  "mysql" ]]; then
         /etc/keystone/keystone.conf
 
     # check to see if table exists already and skip init if so
-    mysql -h ${mysql_host} -u ${mysql_user} -p${mysql_pass} -e "desc ${mysql_db}.migrate_version" &> /dev/null
+    mysql -h "${mysql_host}" -u "${mysql_user}" -p"${mysql_pass}" -e "desc ${mysql_db}.migrate_version" &> /dev/null
     if [[ $? -eq 0 ]]; then
         echo "MySQL database has already been initialized, skipping..."
     else

kibana/start.sh

@@ -2,8 +2,8 @@
 
 python /template.py /kibana.yml.j2 /opt/kibana/config/kibana.yml
 
-if [ $MONASCA_PLUGIN_ENABLED == True ]; then
-    /wait-for.sh $KEYSTONE_URI -- kibana
+if [ "$MONASCA_PLUGIN_ENABLED" == True ]; then
+    /wait-for.sh "$KEYSTONE_URI" -- kibana
 else
     kibana
 fi

kibana/wait-for.sh

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 #   Use this script to test if a given TCP host/port are available
 
-cmdname=$(basename $0)
+cmdname=$(basename "$0")
 
 echoerr() { if [[ $QUIET -ne 1 ]]; then echo "$@" 1>&2; fi }
 
@@ -33,10 +33,10 @@ wait_for()
     while :
     do
         if [[ $ISBUSY -eq 1 ]]; then
-            nc -z $HOST $PORT
+            nc -z "$HOST" "$PORT"
             result=$?
         else
-            (echo > /dev/tcp/$HOST/$PORT) >/dev/null 2>&1
+            (echo > /dev/tcp/"$HOST"/"$PORT") >/dev/null 2>&1
             result=$?
         fi
         if [[ $result -eq 0 ]]; then
@@ -53,9 +53,9 @@ wait_for_wrapper()
 {
     # In order to support SIGINT during timeout: http://unix.stackexchange.com/a/57692
     if [[ $QUIET -eq 1 ]]; then
-        timeout $BUSYTIMEFLAG $TIMEOUT $0 --quiet --child --host=$HOST --port=$PORT --timeout=$TIMEOUT &
+        timeout "$BUSYTIMEFLAG" "$TIMEOUT" "$0" --quiet --child --host="$HOST" --port="$PORT" --timeout="$TIMEOUT" &
     else
-        timeout $BUSYTIMEFLAG $TIMEOUT $0 --child --host=$HOST --port=$PORT --timeout=$TIMEOUT &
+        timeout "$BUSYTIMEFLAG" "$TIMEOUT" "$0" --child --host="$HOST" --port="$PORT" --timeout="$TIMEOUT" &
     fi
     PID=$!
     trap "kill -INT -$PID" INT
@@ -142,7 +142,7 @@ CHILD=${CHILD:-0}
 QUIET=${QUIET:-0}
 
 # check to see if timeout is from busybox?
-TIMEOUT_PATH=$(realpath $(which timeout))
+TIMEOUT_PATH=$(realpath "$(which timeout)")
 if [[ $TIMEOUT_PATH =~ "busybox" ]]; then
         ISBUSY=1
         BUSYTIMEFLAG="-t"
@@ -170,7 +170,7 @@ if [[ $CLI != "" ]]; then
         echoerr "$cmdname: strict mode, refusing to execute subprocess"
         exit $RESULT
     fi
-    exec $CLI
+    exec "$CLI"
 else
-    exit $RESULT
+    exit "$RESULT"
 fi