Page MenuHomePhabricator

add Nemo_bis to jenkins trusted users
Closed, ResolvedPublic

Description

on https://gerrit.wikimedia.org/r/#/c/107376/3

a planet change that touches an erb template, you can see
how jenkins verifies the change, Verified+1

then I do a Code-Review+2 on it and I'm getting "Needs verified" instead of a merge. apparently because i just got a +1 Verified from jenkins but not a +2.

is this intended? then I'd have to hit manual Verified+2 as a human here to merge

03:13 < hashar> how the hell doesn't it not vote +2
03:13 < mutante> yea, that is the question:) not the planet change itself at all

03:17 < hashar> if you could fill a bug about it, that would be nice :-]

this is that bug now

So after thinking about it, is this because NemoBis has created the change and he is not on that "trusted user" regex? I remember we had something similar or even the exact same thing before with a change made by odder. And in the end odder has been added to a regex. How would we handle those if that's what it is?


Version: wmf-deployment
Severity: enhancement

Details

Reference
bz60082

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:56 AM
bzimport set Reference to bz60082.

See https://gerrit.wikimedia.org/r/#/c/92773/ for the change I've been whitelisted in, and the beautiful regex that does that.

hehe, thanks, yea, +1 for "beautiful" :)

So that is definitely confusing but that is the 'check' pipeline which has been invoked and that vote +1, it is because Federico "NemoBis" Leva is not whitelisted ..

Assigning to Nemo since he has the hacking skills required to fix up by adding self to integration/zuul-config.git :-]

Will merge and deploy whenever I see the commit!

Change 110202 had a related patch set uploaded by Nemo bis:
Add Nemo_bis to jenkins trusted users

https://gerrit.wikimedia.org/r/110202

Change 110202 merged by jenkins-bot:
Add Nemo_bis to jenkins trusted users

https://gerrit.wikimedia.org/r/110202

Deployed on Zuul server. Please ping me / reopen this bug if it is not working :-]