vdsm_pep8_gerrit is broken

2013-12-24 Thread Nir Soffer
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. Nir ___ Infra mai

Re: vdsm_pep8_gerrit is broken

2013-12-24 Thread Kiril Nesenko
What pep8 version are you using ? - Kiril - Original Message - > From: "Nir Soffer" > To: infra@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 intenti

Re: vdsm_pep8_gerrit is broken

2013-12-24 Thread Nir Soffer
- Original Message - > From: "Kiril Nesenko" > To: "Nir Soffer" > Cc: infra@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.fedorapeop

Re: vdsm_pep8_gerrit is broken

2013-12-24 Thread Dan Kenigsberg
On Tue, Dec 24, 2013 at 04:47:07PM -0500, Nir Soffer wrote: > - Original Message - > > From: "Kiril Nesenko" > > To: "Nir Soffer" > > Cc: infra@ovirt.org > > Sent: Tuesday, December 24, 2013 11:33:24 PM > > Subject: Re: vdsm_pep8_gerr

Re: vdsm_pep8_gerrit is broken

2013-12-24 Thread Kiril Nesenko
- Original Message - > From: "Dan Kenigsberg" > To: "Nir Soffer" > Cc: "Kiril Nesenko" , infra@ovirt.org > Sent: Wednesday, December 25, 2013 12:31:00 AM > Subject: Re: vdsm_pep8_gerrit is broken > > On Tue, Dec 24, 2013 at 04:

Re: vdsm_pep8_gerrit is broken

2013-12-24 Thread Yaniv Bronheim
- Original Message - > From: "Kiril Nesenko" > To: "Dan Kenigsberg" > Cc: infra@ovirt.org > Sent: Wednesday, December 25, 2013 8:54:49 AM > Subject: Re: vdsm_pep8_gerrit is broken > > > > - Original Message - > > From: &