Last modified: 2014-01-30 09:18:55 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T62082, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60082 - add Nemo_bis to jenkins trusted users
add Nemo_bis to jenkins trusted users
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
wmf-deployment
All All
: Unprioritized enhancement (vote)
: ---
Assigned To: Nemo
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-01-15 11:22 UTC by Daniel Zahn
Modified: 2014-01-30 09:18 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Daniel Zahn 2014-01-15 11:22:13 UTC
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?
Comment 1 Tomasz W. Kozlowski 2014-01-15 12:08:39 UTC
See https://gerrit.wikimedia.org/r/#/c/92773/ for the change I've been whitelisted in, and the beautiful regex that does that.
Comment 2 Daniel Zahn 2014-01-15 12:35:35 UTC
hehe, thanks, yea, +1 for "beautiful" :)
Comment 3 Antoine "hashar" Musso (WMF) 2014-01-15 13:06:28 UTC
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 ..
Comment 4 Antoine "hashar" Musso (WMF) 2014-01-15 13:12:40 UTC
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!
Comment 5 Gerrit Notification Bot 2014-01-29 16:52:59 UTC
Change 110202 had a related patch set uploaded by Nemo bis:
Add Nemo_bis to jenkins trusted users

https://gerrit.wikimedia.org/r/110202
Comment 6 Gerrit Notification Bot 2014-01-30 09:17:50 UTC
Change 110202 merged by jenkins-bot:
Add Nemo_bis to jenkins trusted users

https://gerrit.wikimedia.org/r/110202
Comment 7 Antoine "hashar" Musso (WMF) 2014-01-30 09:18:55 UTC
Deployed on Zuul server. Please ping me / reopen this bug if it is not working :-]

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links