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