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. > > - 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
