Re: Invalid version of pep8 on some slaves
--KJvkvZqQCzHgjKcr
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On 08/21, Nir Soffer wrote:
Hi infra,
=20
There is too old version of pep8 used on some slave, that cause pep8 test=
to fail
for
valid sources.
=20
Here is one example:
=20
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/11846/console
=20
gerrit failure:
=20
+ pep8 --version
1.4.5
+ git diff HEAD~
+ sed 's%^./*%%'
+ pep8 --diff '--filename=3D*.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 i=
ndentation of opening bracket's line
=20
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
Can you please supply the correct version so we can add it to our
repos?
Thanks!
=20
$ pep8 lib/vdsm/tool/configurator.py; echo $?
0
=20
Thanks,
Nir
_______________________________________________
Infra mailing list
Infra(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra
--=20
David Caro
Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D
Tel.: +420 532 294 605
Email: dcaro(a)redhat.com
Web: www.redhat.com
RHT Global #: 82-62605
--KJvkvZqQCzHgjKcr
Content-Type: application/pgp-signature
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iQEcBAEBAgAGBQJT9ikgAAoJEEBxx+HSYmnDR68IAIz5OuX4tYt3vrONWVLsovbE
DZbpPpYQSzhVu7YOEsK0HyZoqRgJth3rmzFvFNH4FCNeTEcwVgSiCq/LlSKuv35P
oIU+bUyo58hfdN2l+n75UgF9tGqN/5tR0MACsHyfNv8KZ0shxn2LwWBsbvUu57uR
GbYrHusQj2bq3oSM/D+fTufoOYTaHDf4re6GCgqTuh5ZarOmq01Au7EAX8OiNEFh
u0QMcONwudl9qVB8cKrxtWkyYPJhJpiZmljqrt9BIaBdPSU9+axrFFwgnlnmTJo6
NXbLRz+/X7qsVlkVkTCrD/OhapsIa4B/1HJHcf0YtM5g3QUG0zjfoJC/TtEaC3c=
=FCLQ
-----END PGP SIGNATURE-----
--KJvkvZqQCzHgjKcr--
Attachments:
- attachment.bin (multipart/signed — 2.0 KB)