On Fri, Jul 31, 2015 at 9:06 AM, Michal Skrivanek < [email protected]> wrote:
> > On Jul 31, 2015, at 08:08 , Sandro Bonazzola <[email protected]> wrote: > > > Hi, > > vdsm from 3.5 branch is currently failing to build on Fedora, failing > pep8. > > Please fix ASAP. > > > > Fedora 20: > http://jenkins.ovirt.org/view/Stable%20branches%20per%20project/view/vdsm/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/288/artifact/exported-artifacts/build.log > > Fedora 21: > http://jenkins.ovirt.org/view/Stable%20branches%20per%20project/view/vdsm/job/vdsm_3.5_create-rpms-fc21-x86_64_merged/114/artifact/exported-artifacts/build.log > > looks like [1] merged 3 weeks ago > > hm..running CI on merged patches doesn't alert on failure? > Looks like no email is triggered on failure since jenkins already comment on gerrit and gerrit send an email to all people reviewing the patch. So all of them have been warned that the patch was failing on merge. I think vdsm is also missing a pep8 / pyflakes validation on patch sent, it would help avoiding to discover the issue after having merged the patch. > > Thanks, > michal > > [1] https://gerrit.ovirt.org/#/c/43276/ > > > > > > > -- > > Sandro Bonazzola > > Better technology. Faster innovation. Powered by community collaboration. > > See how it works at redhat.com > > _______________________________________________ > > Devel mailing list > > [email protected] > > http://lists.ovirt.org/mailman/listinfo/devel > > -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com
_______________________________________________ Devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/devel
