]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Don't do PEP8 test for openstack-common code.
authorZhiteng Huang <zhiteng.huang@intel.com>
Mon, 23 Jul 2012 06:39:48 +0000 (14:39 +0800)
committerZhiteng Huang <zhiteng.huang@intel.com>
Mon, 23 Jul 2012 06:39:48 +0000 (14:39 +0800)
Pep8 test reports issue on openstack-common modules, which would
fail CI test. Pep8 test should skip openstack-common and assuming
they are problem-free code. Even it's not, bug/fix should go to
openstack-common first, then resync back to Cinder.  Therefore,
Cinder should skip pep8 test for openstack-common code.

Fix bug #1027774

Change-Id: I22c45542401022a6bad9f2320e8657f972915f69

run_tests.sh

index c08eb6a345fdf9886b613c8adb193e40084c5605..4e64ec90df5ff9e7d15798208d0de971093be8ed 100755 (executable)
@@ -104,7 +104,7 @@ function run_tests {
 # NOTE(dprince): Exclude xenapi plugins. They are Python 2.4 code and as such
 #                cannot be expected to work with tools/hacking checks.
 xen_net_path="plugins/xenserver/networking/etc/xensource/scripts"
-srcfiles=`find cinder -type f -name "*.py"`
+srcfiles=`find cinder -type f -name "*.py" ! -path "cinder/openstack/common/*"`
 srcfiles+=" `find bin -type f ! -name "cinder.conf*" ! -name "*api-paste.ini*"`"
 srcfiles+=" `find tools -type f -name "*.py"`"
 srcfiles+=" setup.py"