I see that i have a list of Outgoing reviews that are Verified +1 git-reviews from jenkins bot. it appears that they have passed build tests but none of them are merged. Did i drop the ball and missed a last step to merge?
Outgoing reviews Verified +1 git-reviews


Requires Code-Review +2 label and Verified +2 label to merge a patch. See https://www.mediawiki.org/wiki/Gerrit/Code_review#Complete_the_review

This is a successful merged case: https://gerrit.wikimedia.org/r/#/c/395768/

Thanks. That what i needed to read.

Once you have been around for a while and a couple of your patches got merged, you can ask someone to add you to the CI whitelist so the bot will add Verified +2 instead of Verified +1. (The difference is that +2 involves some tests that can be abused by a malicious patch uploader.)

Can I join the CI whitelist in my situation?

Well, I don’t see where one can ask to get added to the CI whitelist, so I’m going to also ask to get on that whitelist

Unfortunately I had very little free time but I wrote some documentation on the whitelist to make this more of a self-service thing.

Thanks for you make time.

This made me wonder… can’t we make it so that +2 is granted automaticly after a few patches of an author have been merged ? A bit like autoconfirmed ?

The grand plan is to make every build run in its own disposable container and get rid of the whitelist, but in the short term something like that would be nice, yeah.

For me, the CI whitelist like confirmed
user group (confirm manually).