Okay and I don't want to beg for 'trusted' status, but this basically means that any C+2able user which isn't trusted can't really C+2 entries. Or does a C+2 start jenkins?
2015-01-15 17:29 GMT+01:00 Fabian Neundorf CommodoreFabianus@gmx.de:
But according to https://gerrit.wikimedia.org/r/184967 it shouldn't trigger them. And okay didn't saw that only trusted accounts would be considered for +1.
And is there no way to have some tests executed even from not trusted accounts or is that security wise impossible?
2015-01-15 13:33 GMT+01:00 Merlijn van Deen valhallasw@arctus.nl:
On 15 January 2015 at 13:02, Fabian Neundorf CommodoreFabianus@gmx.de wrote:
And just for clarification, when someone uploads a patch, not the e-mail in the git commit are important but the account which you used to push it to the repository?
Yes, the account used to submit the Gerrit changeset.
Couldn't you automatically generate a list automatically from the admins (or +2able contributors?) of the project?
And why is it okay to run the test on a CR+1 vote (which is afaik everyone with an account) but not on push?
The test are triggered on a +1 from /whitelisted accounts/.
The list of whitelisted accounts is here: https://github.com/wikimedia/integration-config/blob/master/zuul/layout.yaml...
As a workaround, posting 'recheck' should also trigger the tests. Antoine, maybe it's an idea to also trigger on comments that start with 'check' (instead of 'only contain recheck')? That might be easier to implement than your +1 solution.
Merlijn
Pywikipedia-l mailing list Pywikipedia-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l