Only build changes that come from @google.com.
Googlers should be able to tell the buildbot to launch builds for an external contributor's change, but Gerrit is determined to make this hard for us. The email sent by Gerrit for comments doesn't actually contain the email address of the commenter in any clear way, only the name. It does however contain a list of `Gerrit-Reviewer: Name <email>` entries that could be used for this though. Change-Id: Iee61f06dcd8f0024f16b535ea9a34765bf01b2e7
This commit is contained in:
parent
d7f935a05b
commit
29be129342
1 changed files with 6 additions and 0 deletions
|
@ -200,6 +200,8 @@ def build_project(gerrit_info, dry_run):
|
|||
|
||||
|
||||
def handle_change(gerrit_info, _, dry_run):
|
||||
if '@google.com' not in gerrit_info['Gerrit-Owner']:
|
||||
return True
|
||||
return build_project(gerrit_info, dry_run)
|
||||
handle_newchange = handle_change
|
||||
handle_newpatchset = handle_change
|
||||
|
@ -230,6 +232,10 @@ def handle_comment(gerrit_info, body, dry_run):
|
|||
if 'Verified+1' in body:
|
||||
drop_rejection(gerrit_info, dry_run)
|
||||
|
||||
# TODO(danalbert): Needs to be based on the account that made the comment.
|
||||
if '@google.com' not in gerrit_info['Gerrit-Owner']:
|
||||
return True
|
||||
|
||||
command_map = {
|
||||
'clean': lambda: clean_project(gerrit_info, dry_run),
|
||||
'retry': lambda: build_project(gerrit_info, dry_run),
|
||||
|
|
Loading…
Reference in a new issue