
21 Aug
2014
21 Aug
'14
7:04 p.m.
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