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