Remove extra output if CI_DEBUG is false

This way we have a nicer view in Jenkins when
preparing Functest

Change-Id: Ie0df3aa8b07ac2fd726d11b1ab01a7291d4c2c5e
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>

3 files changed, 15 insertions, 6 deletions

diff --git a/docker/run_tests.sh b/docker/run_tests.sh
index 4fd505bb..f4e17888 100755
--- a/docker/run_tests.sh
+++ b/docker/run_tests.sh
@@ -40,15 +40,19 @@ else
 fi
 
 function clean_openstack(){
+    echo -e "\n\nCleaning Openstack environment..."
     python ${FUNCTEST_REPO_DIR}/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py \
         --debug
+    echo -e "\n\n"
 }
 
 function run_test(){
     test_name=$1
+    echo ""
     echo "----------------------------------------------"
-    echo "------------- Running $i test case  "
+    echo "  Running test case: $i"
     echo "----------------------------------------------"
+    echo ""
     case $test_name in
         "vping")
             info "Running vPing test..."
diff --git a/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py b/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py
index eab89d22..866e3a82 100644
--- a/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py
+++ b/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py
@@ -69,7 +69,7 @@ default_users = defaults_yaml.get(installer).get("users")
 default_tenants = defaults_yaml.get(installer).get("tenants")
 
 def separator():
-    logger.debug("-------------------------------------------")
+    logger.info("-------------------------------------------")
 
 def remove_instances(nova_client):
     logger.info("Removing Nova instances...")
@@ -375,7 +375,6 @@ def main():
         logger.error("Please source the openrc credentials and run the script again.")
         exit(-1)
 
-
     remove_instances(nova_client)
     separator()
     remove_images(nova_client)
diff --git a/testcases/config_functest.py b/testcases/config_functest.py
index 11682580..199fadfd 100755
--- a/testcases/config_functest.py
+++ b/testcases/config_functest.py
@@ -125,7 +125,10 @@ def action_start():
         script += 'bundle install'
 
         cmd = "/bin/bash -c '" + script + "'"
-        functest_utils.execute_command(cmd,logger)
+        if os.environ.get("CI_DEBUG") == "false":
+            functest_utils.execute_command(cmd)
+        else:
+            functest_utils.execute_command(cmd,logger)
 
         # Create result folder under functest if necessary
         if not os.path.exists(RALLY_RESULT_DIR):
@@ -208,7 +211,7 @@ def action_clean():
 
     logger.debug("Cleaning up the OpenStack deployment...")
     cmd='python ' + REPO_PATH + \
-        '/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py -d '
+        '/testcases/VIM/OpenStack/CI/libraries/clean_openstack.py'
     functest_utils.execute_command(cmd,logger)
     logger.info("Functest environment clean!")
 
@@ -221,7 +224,10 @@ def install_rally():
         logger.debug("Executing %s/install_rally.sh..." %RALLY_REPO_DIR)
         install_script = RALLY_REPO_DIR + "/install_rally.sh --yes"
         cmd = 'sudo ' + install_script
-        functest_utils.execute_command(cmd,logger)
+        if os.environ.get("CI_DEBUG") == "false":
+            functest_utils.execute_command(cmd)
+        else:
+            functest_utils.execute_command(cmd,logger)
 
         logger.debug("Creating Rally environment...")
         cmd = "rally deployment create --fromenv --name="+DEPLOYMENT_MAME