[TICKET] gerrit marked patch as still -1, but jenkins supposedly removed the -1

Eyal Edri eedri at redhat.com
Wed May 27 13:03:04 UTC 2015



----- Original Message -----
> From: "Eyal Edri" <eedri at redhat.com>
> To: "Greg Sheremeta" <gshereme at redhat.com>
> Cc: "infra" <infra at ovirt.org>
> Sent: Tuesday, May 26, 2015 7:35:50 PM
> Subject: Re: [TICKET] gerrit marked patch as still -1, but jenkins	supposedly	removed the -1
> 
> 
> 
> ----- Original Message -----
> > From: "Greg Sheremeta" <gshereme at redhat.com>
> > To: "infra" <infra at ovirt.org>
> > Sent: Tuesday, May 26, 2015 7:25:05 PM
> > Subject: [TICKET] gerrit marked patch as still -1, but jenkins supposedly
> > 	removed the -1
> > 
> > Why is this patch (which is merged!) -1 verified from jenkins?
> > 
> > https://gerrit.ovirt.org/#/c/41296/
> > 
> > (May be related to the bug I've already reported on Jenkins / gerrit
> > integration)
> 
> it looks like jenkins isn't dropping the -1 from verified even if a second
> build successeded.
> i changed the gerrit settings so it will give +1 on verify to see if that's
> the case,
> so now you will see +1 verify and +code review, if we don't want ci to
> +verify
> then we can just not use the verify flag and stick to the code review which
> looks to be working.
> 

can you check if this still happens? after i changed the settings.

> e.
> 
> > 
> > 
> > Greg Sheremeta
> > Red Hat, Inc.
> > Sr. Software Engineer, RHEV
> > Cell: 919-807-1086
> > gshereme at redhat.com
> > _______________________________________________
> > Infra mailing list
> > Infra at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/infra
> > 
> > 
> > 
> _______________________________________________
> Infra mailing list
> Infra at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/infra
> 
> 
> 



More information about the Infra mailing list