Bad pep8 on slave?

Dan Kenigsberg danken at redhat.com
Tue Feb 18 11:38:25 UTC 2014


On Mon, Feb 17, 2014 at 05:47:58PM -0500, Nir Soffer wrote:
> Hi infra,
> 
> Note this strange failure of pep8:
> http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6390/

The following failure should have been hacked by http://gerrit.ovirt.org/24602 .
Could you rebase your code?

/usr/bin/pep8 --version
1.4.5
/usr/bin/pep8 --exclude="config.py,constants.py" --filename '*.py,*.py.in' \
        client lib/vdsm/*.py lib/vdsm/*.py.in tests vds_bootstrap vdsm-tool vdsm/*.py vdsm/*.py.in vdsm/netconf vdsm/sos/vdsm.py.in vdsm/storage vdsm/vdsm vdsm_api vdsm_hooks vdsm_reg 
lib/vdsm/netlink.py:158:9: E123 closing bracket does not match indentation of opening bracket's line

> 
> While the pep8 job succeeds:
> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/7174/

Kiril, would you add a `pep8 --version` call to that script, so we can tell
which version it was using?

> 
> Can you check why this happens?
> 
> What is the difference between vdsm_unit_tests_gerrit_el and vdsm_unit_tests_gerrit?

I believe that the first runs on an el6 slave, and the latter on a Fedora.



More information about the Infra mailing list