
27 Dec
2013
27 Dec
'13
10:02 a.m.
Current version checks only .py files, skipping errors in py.in files, vdsm and supervdsmServer scripts. Here is the script needed to fix this issue: rm -f ${WORKSPACE}/violations.txt pushd $WORKSPACE/vdsm git diff HEAD~ | pep8 --diff --filename '*.py,*.py.in,vdsm,supervdsmServer' | sed 's%^./*%%' > ${WORKSPACE}/violations.txt if [[ ${PIPESTATUS[1]} -ne 0 ]]; then echo "PEP8 Failed" cat ${WORKSPACE}/violations.txt else echo "PEP8 is ok" fi For testing this, trigger this job with patch http://gerrit.ovirt.org/22724 - you should see failure vdsm, supervdsmServer, bad.py, bad.py.in