[Adding David]

David,
Can you access the slave and update pep8 there ?

- Kiril

----- Original Message -----
> From: "Dan Kenigsberg" <[email protected]>
> To: "Mark Wu" <[email protected]>, "Kiril Nesenko" 
> <[email protected]>
> Cc: [email protected]
> Sent: Monday, April 29, 2013 1:43:13 PM
> Subject: vdsm does not build on fedora17
> 
> On Mon, Apr 29, 2013 at 04:46:18AM +0100, Jenkins ci oVirt Server wrote:
> > Project: http://jenkins.ovirt.org/job/vdsm_create_rpms/
> > Build: http://jenkins.ovirt.org/job/vdsm_create_rpms/365/
> 
> Hi Mark, Kiril,
> 
> Since I've taken http://gerrit.ovirt.org/14198 in, we're getting pep8
> error on fedora17 slave
> http://jenkins.ovirt.org/job/vdsm_create_rpms/label=fedora17/365/consoleFull
> 
> /bin/pep8 --exclude="config.py,constants.py" --filename '*.py,*.py.in' \
>     client lib/betterPopen/*.py 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
> vdsm/configNetwork.py:562:56: E251 no spaces around keyword / parameter
> equals
> 
> I'm guessing that this can be fixed by updating pep8 on the fedora17 slave as
> the issue is not caught by other slaves.
> 
> Kiril, could you do this upgrade?
> 
> Dan.
> 
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to