Jenkins CI granting excessive +1s

Dan Kenigsberg danken at redhat.com
Wed Jun 17 13:01:06 UTC 2015


On Tue, Jun 16, 2015 at 08:48:17PM +0200, dcaro wrote:
> On 06/16, Eyal Edri wrote:
> > Sounds like a bug.
> > Jenkins should only use the CI flag and not change CR and V flags.
> > 
> > Might have been a misunderstanding.
> > David, can you fix it so Jenkins will only update CI flag?
> 
> Can someone pass me an example of the patch that had the reviews? It's possible
> that previously the jobs were manually modified to only set the ci flag and we
> updated them on yaml and the manual changes got reverted. Having a sample patch
> would allow me to trim down the list of possible jobs that give that review.
> 
> 
> That can be prevented globally easily, but until all projects have the ci flag
> we can't enforce it globally.

See for example https://gerrit.ovirt.org/#/c/42362/

Patch Set 5: Code-Review+1 Continuous-Integration+1 Verified+1

in a later run seems to have fixed this

Patch Set 6: Continuous-Integration+1



More information about the Infra mailing list