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(a)arctus.nl>nl>:
On 15 January 2015 at 13:02, Fabian Neundorf
<CommodoreFabianus(a)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.yam…
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(a)lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l