vdsm_pep8_gerrit is broken

[Dan Kenigsberg]

On Tue, Dec 24, 2013 at 04:47:07PM -0500, Nir Soffer wrote:
> ----- Original Message -----
> > From: "Kiril Nesenko" <knesenko at redhat.com>
> > To: "Nir Soffer" <nsoffer at redhat.com>
> > Cc: infra at ovirt.org
> > Sent: Tuesday, December 24, 2013 11:33:24 PM
> > Subject: Re: vdsm_pep8_gerrit is broken
> > 
> > What pep8 version are you using ?
> 
> We use http://danken.fedorapeople.org/python-pep8-1.4.5-2.el6.noarch.rpm
> 
> You can check what we use here: http://www.ovirt.org/Vdsm_Developers#Installing_required_packages
> 
> > ----- Original Message -----
> > > From: "Nir Soffer" <nsoffer at redhat.com>
> > > To: infra at ovirt.org
> > > Sent: Tuesday, December 24, 2013 11:27:54 PM
> > > Subject: vdsm_pep8_gerrit is broken
> > > 
> > > Hi infra,
> > > 
> > > I added a patch [1], breaking pep8 intentionally - it is passed the pep8
> > > job
> > > [2]
> > > 
> > > [1] http://gerrit.ovirt.org/22724
> > > [2] http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/6260/
> > > 
> > > This explains why lately jenkins was rather liberal.

Kiril, could you remind me where the code for the Jenkins jobs can be
seen? Does it have it's own gerrit repo? I'd like vdsm_pep8_gerrit to
call `pep8 --version`.

Nir, I suspect that `pep --diff` simply ignores changes in files lacking
the *py suffix.