----- Original Message -----
From: "David Caro" <dcaroest@redhat.com> To: "Nir Soffer" <nsoffer@redhat.com> Cc: "infra" <infra@ovirt.org> Sent: Thursday, August 21, 2014 8:15:12 PM Subject: Re: Invalid version of pep8 on some slaves
On 08/21, Nir Soffer wrote:
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):
That's a centos 6 slave, and it seens that's the greatest python-pep8 we have for centos, it's greater than the on here: http://www.ovirt.org/Vdsm_Developers
David: you are correct, we need to fix the documentation. Dan: the version on that wiki page is too old - do you have a newer build?
Can you please supply the correct version so we can add it to our repos?
Thanks!
$ pep8 lib/vdsm/tool/configurator.py; echo $? 0
Thanks, Nir _______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra
-- David Caro
Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R&D
Tel.: +420 532 294 605 Email: dcaro@redhat.com Web: www.redhat.com RHT Global #: 82-62605