Invalid version of pep8 on some slaves

Nir Soffer nsoffer at redhat.com
Thu Aug 21 17:04:44 UTC 2014


Hi infra,

There is too old version of pep8 used on some slave, that cause pep8 test to fail for
valid sources.

Here is one example:

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/11846/console

gerrit failure:

+ pep8 --version
1.4.5
+ git diff HEAD~
+ sed 's%^./*%%'
+ pep8 --diff '--filename=*.py,*.py.in,*/vdsm,*/supervdsmServer'
+ [[ 1 -ne 0 ]]
+ echo 'PEP8 Failed'
PEP8 Failed
+ cat /home/jenkins/workspace/vdsm_master_pep8_gerrit/violations.xml
lib/vdsm/tool/configurator.py:41:9: E123 closing bracket does not match indentation of opening bracket's line

Running pep8 on Fedora 19 (version 1.4.6):

$ pep8 lib/vdsm/tool/configurator.py; echo $?
0

Thanks,
Nir



More information about the Infra mailing list