----- Original Message ----- > From: "Yaniv Bronheim" <[email protected]> > To: "Kiril Nesenko" <[email protected]> > Cc: "Dan Kenigsberg" <[email protected]>, "Ewoud Kohl van Wijngaarden" > <[email protected]>, > [email protected] > Sent: Tuesday, April 30, 2013 11:41:40 AM > Subject: Re: vdsm does not build on fedora17 > > the version is alright and the error seems meaningful if the spaces are not > there > > what branch of vdsm do you run there? > master.
Seems like this patch - http://gerrit.ovirt.org/#/c/14198/ causes build to fail. > > ----- Original Message ----- > > From: "Kiril Nesenko" <[email protected]> > > To: "Dan Kenigsberg" <[email protected]> > > Cc: "Ewoud Kohl van Wijngaarden" <[email protected]>, > > [email protected] > > Sent: Tuesday, April 30, 2013 11:08:31 AM > > Subject: Re: vdsm does not build on fedora17 > > > > > > > > ----- Original Message ----- > > > From: "Ewoud Kohl van Wijngaarden" <[email protected]> > > > To: "Kiril Nesenko" <[email protected]> > > > Cc: "David Caro" <[email protected]>, "Dan Kenigsberg" > > > <[email protected]>, [email protected], > > > "Mark Wu" <[email protected]> > > > Sent: Monday, April 29, 2013 7:13:38 PM > > > Subject: Re: vdsm does not build on fedora17 > > > > > > On Mon, Apr 29, 2013 at 11:02:43AM -0400, Kiril Nesenko wrote: > > > > > > > > ----- Original Message ----- > > > > > From: "Ewoud Kohl van Wijngaarden" <[email protected]> > > > > > To: "David Caro" <[email protected]> > > > > > Cc: "Kiril Nesenko" <[email protected]>, "Dan Kenigsberg" > > > > > <[email protected]>, [email protected], > > > > > "Mark Wu" <[email protected]> > > > > > Sent: Monday, April 29, 2013 4:20:03 PM > > > > > Subject: Re: vdsm does not build on fedora17 > > > > > > > > > > On Mon, Apr 29, 2013 at 03:05:32PM +0200, David Caro wrote: > > > > > > On 04/29, Kiril Nesenko wrote: > > > > > > > [Adding David] > > > > > > > > > > > > > > David, > > > > > > > Can you access the slave and update pep8 there ? > > > > > > > > > > > > No, that server is owned by ekohl, adding him to the thread. > > > > > > > > > > Update to which version? I've done a yum update python-pep8 but that > > > > > shows no possible updates. > > > > > > > > > > > > > Can you post here the version of python-pep8 you have ? > > > > > > yum tells me 1.0.1-1.fc17. > > > > Dan ^^ ? > > > > > > > > > > > > > > > > > > > > > > > - 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. > > > > > > > > > > > > > > > > > > > > -- > > > > > > David Caro > > > > > > > > > > > > Red Hat Czech s.r.o. > > > > > > Continuous Integration Engineer - EMEA ENG Virtualization R&D > > > > > > > > > > > > Tel.: +420 532 294 605 > > > > > > Email: [email protected] > > > > > > Web: www.cz.redhat.com > > > > > > Red Hat Czech s.r.o., Purkyňova 99/71, 612 45, Brno, Czech Republic > > > > > > RHT Global #: 82-62605 > > > > > > > > > > _______________________________________________ > > vdsm-patches mailing list > > [email protected] > > https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches > > > _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
