Page MenuHomePhabricator

jenkins now uses the author of the commit instead of the committer when deciding whether to run the unit tests
Closed, ResolvedPublic

Description

jenkins now uses the author of the commit instead of the committer when deciding whether to run the unit tests, or at least that's how it seems. This makes it impossible for "trusted" people to make it run unit tests of somebody else's code. See e.g. PS6 of https://gerrit.wikimedia.org/r/#/c/97307/ .


Version: unspecified
Severity: normal

Details

Reference
bz60769

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:06 AM
bzimport set Reference to bz60769.
bzimport added a subscriber: Unknown Object (MLST).

I have enabled Gerrit debugging on Zuul server. Will look at it later tonight.

This is caused by a regression in Gerrit 2.8.1. See bug 60781

That was fixed, this should be good now too.