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