Add venv wrapper for check_uptodate.sh

tools/conf/check_uptodate.sh is added in run_tests.sh

For simple, run "run_tests.sh -p" will call it, and get following error:

Failed to collect options from module
cinder.volume.drivers.san.hp.hp_3par_common: No module named hp3parclient

Actually, the hp3parclient is listed in test-requirements.txt, and it is
installed in venv.

the check_uptodate.sh should run in venv to get correct result.

update the error message to update the cindier.conf.sample correctly.

-----------
Test method
-----------
$ pwd
/opt/stack/cinder

$ source .venv/bin/activate; tools/conf/generate_sample.sh

(.venv)$ deactivate

$ ./run_tests.sh  -p
Running flake8 ...
// no error emitted here

Fixes Bug #1219197

Change-Id: I9b3eaf3c1a5eb395eefed28bb907685cdef147aa

diff --git a/run_tests.sh b/run_tests.sh
index 6e9e17d852b1553b6a29e3755acb3e8b48affcca..ae6e365bbf53540f4fca39f1b65060b4af430f21 100755 (executable)
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -227,7 +227,7 @@ fi
 
 if [ $just_pep8 -eq 1 ]; then
     run_pep8
-    bash ./tools/conf/check_uptodate.sh
+    ${wrapper}   bash ./tools/conf/check_uptodate.sh
     exit
 fi
 
@@ -245,6 +245,6 @@ run_tests
 if [ -z "$testrargs" ]; then
   if [ $no_pep8 -eq 0 ]; then
     run_pep8
-    bash ./tools/conf/check_uptodate.sh
+    ${wrapper} bash ./tools/conf/check_uptodate.sh
   fi
 fi

diff --git a/tools/conf/check_uptodate.sh b/tools/conf/check_uptodate.sh
index df9bc74c5b25118635438e34289bef3551f91c84..bfe199b456bcb91a1fc036fb7854bcc599667e48 100755 (executable)
--- a/tools/conf/check_uptodate.sh
+++ b/tools/conf/check_uptodate.sh
@@ -4,7 +4,8 @@ CFGFILE=cinder.conf.sample
 tools/conf/generate_sample.sh -o $TEMPDIR
 if ! diff $TEMPDIR/$CFGFILE etc/cinder/$CFGFILE
 then
-    echo "E: cinder.conf.sample is not up to date, please run tools/conf/generate_sample.sh"
+    echo "E: cinder.conf.sample is not up to date, please run tools/conf/generate_sample.sh in venv"
+    echo "E: e.g. \$ source .venv/bin/activate; tools/conf/generate_sample.sh"
     rm -rf $TEMPDIR
     exit 42
 fi