Merge "fix 'unary operator expected' in conditional expression"

diff --cc ci/deploy.sh
index 55ba5b6,3ce14c2..3143aae
--- 1/ci/deploy.sh
--- 2/ci/deploy.sh
+++ b/ci/deploy.sh
@@@ -788,21 -788,20 +788,21 @@@ function undercloud_prep_overcloud_depl
        DEPLOY_OPTIONS+=" -e /usr/share/openstack-tripleo-heat-templates/environments/opendaylight.yaml"
      fi
      SDN_IMAGE=opendaylight
-     if [ ${deploy_options_array['sfc']} == 'true' ]; then
+     if [ "${deploy_options_array['sfc']}" == 'true' ]; then
        SDN_IMAGE+=-sfc
      fi
-   elif [ ${deploy_options_array['sdn_controller']} == 'opendaylight-external' ]; then
+   elif [ "${deploy_options_array['sdn_controller']}" == 'opendaylight-external' ]; then
      DEPLOY_OPTIONS+=" -e /usr/share/openstack-tripleo-heat-templates/environments/opendaylight-external.yaml"
      SDN_IMAGE=opendaylight
-   elif [ ${deploy_options_array['sdn_controller']} == 'onos' ]; then
+   elif [ "${deploy_options_array['sdn_controller']}" == 'onos' ]; then
      DEPLOY_OPTIONS+=" -e /usr/share/openstack-tripleo-heat-templates/environments/onos.yaml"
      SDN_IMAGE=opendaylight
-   elif [ ${deploy_options_array['sdn_controller']} == 'opencontrail' ]; then
+   elif [ "${deploy_options_array['sdn_controller']}" == 'opencontrail' ]; then
      echo -e "${red}ERROR: OpenContrail is currently unsupported...exiting${reset}"
      exit 1
-   elif [[ -z ${deploy_options_array['sdn_controller']} || ${deploy_options_array['sdn_controller']} == 'false' ]]; then
+   elif [[ -z "${deploy_options_array['sdn_controller']}" || "${deploy_options_array['sdn_controller']}" == 'false' ]]; then
      echo -e "${blue}INFO: SDN Controller disabled...will deploy nosdn scenario${reset}"
 +    SDN_IMAGE=opendaylight
    else
      echo "${red}Invalid sdn_controller: ${deploy_options_array['sdn_controller']}${reset}"
      echo "${red}Valid choices are opendaylight, opendaylight-external, onos, opencontrail, false, or null${reset}"