>> >> http://jenkins.ovirt.org/job/ovirt-engine_3.6_check-patch-el6-x86_64/1312/ : >> FAILURE >> >> http://jenkins.ovirt.org/job/ovirt-engine_3.6_check-patch-el7-x86_64/1410/ : >> FAILURE > > Both of these are pep8: > > packaging/setup/plugins/ovirt-engine-rename/ovirt-engine/database.py:311:21: > E126 continuation line over-indented for hanging indent > packaging/setup/plugins/ovirt-engine-rename/ovirt-engine/database.py:312:21: > E126 continuation line over-indented for hanging indent > >> >> http://jenkins.ovirt.org/job/ovirt-engine_3.6_check-patch-fc23-x86_64/1004/ >> : SUCCESS > > How come this one passed? > > Do we have different versions/configuration of pep8 in different OSes?
If you didn't do anything special, you are probably using PEP8 from the distro repositories, which is different across repos... If you want to use the same pep8 on every distro, you will need to modify check_patch.sh to do that. For example you could make it pull a specific PEP8 version with pip. > > Also note that E126 is ignored by default, see e.g.: > > https://github.com/PyCQA/pep8/issues/265 > https://github.com/PyCQA/pep8/pull/360 > This was probably added on F23 but not landed in EL6/7 yey (or will not ever). You can force this my making pep8 used some configuration file in check_patch.sh -- Barak Korren bkor...@redhat.com RHEV-CI Team _______________________________________________ Infra mailing list Infra@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra