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