[Contrail 3.0.1] Plugin tests did not pass shellcheck

Bug #1570904 reported by Yevgeniy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel Contrail plugin
Fix Released
Medium
Yevgeniy

Bug Description

Scenario:
  1. Install shellcheck
Actual result:

In utils/jenkins/system_tests.sh line 26:
if [ -z $NODE_VOLUME_SIZE ]; then export NODE_VOLUME_SIZE=350; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 27:
if [ -z $OPENSTACK_RELEASE ]; then export OPENSTACK_RELEASE=Ubuntu; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 28:
if [ -z $ENV_NAME ]; then export ENV_NAME="contrail"; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 29:
if [ -z $ADMIN_NODE_MEMORY ]; then export ADMIN_NODE_MEMORY=4096; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 30:
if [ -z $ADMIN_NODE_CPU ]; then export ADMIN_NODE_CPU=4; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 31:
if [ -z $SLAVE_NODE_MEMORY ]; then export SLAVE_NODE_MEMORY=4096; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 32:
if [ -z $SLAVE_NODE_CPU ]; then export SLAVE_NODE_CPU=4; fi
        ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 281:
    ISO="`ls ${WORKSPACE}/build/iso/*.iso | head -n 1`"
         ^-- SC2006: Use $(..) instead of deprecated `..`
          ^-- SC2012: Use find instead of ls to better handle non-alphanumeric filenames.
             ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 369:
                echo wget -c ${ISO_URL} -O ${ISO_PATH}
                             ^-- SC2086: Double quote to prevent globbing and word splitting.
                                           ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 372:
                wget -c ${ISO_URL} -O ${ISO_PATH}
                        ^-- SC2086: Double quote to prevent globbing and word splitting.
                                      ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 404:
      mkdir -p $LOGS_DIR
               ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 416:
        if [ $(dos.py list | grep "^${ENV_NAME}\$") ]; then
             ^-- SC2046: Quote this to prevent word splitting.
             ^-- SC2143: Instead of [ -n $(foo | grep bar) ], use foo | grep -q bar .
In utils/jenkins/system_tests.sh line 437:
        echo python plugin_test/run_tests.py -q --nologcapture --with-xunit ${OPTS}
                                                                            ^-- SC2086: Double quote to prevent globbing and word splitting
In utils/jenkins/system_tests.sh line 440:
        echo ${PYTHONPATH}
             ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 441:
        python plugin_test/run_tests.py -q --nologcapture --with-xunit ${OPTS}
                                                                       ^-- SC2086: Double quote to prevent globbing and word splitting.
In utils/jenkins/system_tests.sh line 481:
GetoptsVariables ${@}
                 ^-- SC2068: Double quote array expansions, otherwise they're like $* and break on spaces.

Expected result:
   no errors occurs

Tags: contrail
Yevgeniy (yshapovalov)
Changed in fuel-plugin-contrail:
importance: Undecided → Medium
tags: added: contrail
Changed in fuel-plugin-contrail:
assignee: nobody → Yevgeniy (yshapovalov)
milestone: none → 3.0.1
status: New → Confirmed
Yevgeniy (yshapovalov)
Changed in fuel-plugin-contrail:
status: Confirmed → In Progress
Yevgeniy (yshapovalov)
Changed in fuel-plugin-contrail:
status: In Progress → Fix Committed
Yevgeniy (yshapovalov)
Changed in fuel-plugin-contrail:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.