diff options
author | Eric Boren <borenet@google.com> | 2018-04-26 13:09:48 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-04-26 17:34:43 +0000 |
commit | 1eec99cf9434f1361c0c355ea0e26e86729df199 (patch) | |
tree | eb22dee89a46d44bbc15709fee95264704468a7c /PRESUBMIT.py | |
parent | 88611a1a717721c7e7aff1d691dd9af850c35259 (diff) |
AUTHORS check: Allow service accounts from skia-swarming-bots
Bug: skia:7833, skia:7050
Change-Id: Id5b3fdd1b0a4ae50fce7a5d1212c744a85274012
Reviewed-on: https://skia-review.googlesource.com/123936
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 0a2aa8e267..94a4bce7b6 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -60,7 +60,9 @@ PATH_PREFIX_TO_EXTRA_TRYBOTS = { # 'src/image/SkImage_Base.h': 'master5:pqr,stu;master1:abc1;master2:def', } -SERVICE_ACCOUNT_SUFFIX = '@skia-buildbots.google.com.iam.gserviceaccount.com' +SERVICE_ACCOUNT_SUFFIX = [ + '@%s.google.com.iam.gserviceaccount.com' % project for project in [ + 'skia-buildbots', 'skia-swarming-bots']] def _CheckChangeHasEol(input_api, output_api, source_file_filter=None): @@ -364,8 +366,9 @@ def _CheckOwnerIsInAuthorsFile(input_api, output_api): owner_email = cr.GetOwnerEmail() # Service accounts don't need to be in AUTHORS. - if owner_email.endswith(SERVICE_ACCOUNT_SUFFIX): - return results + for suffix in SERVICE_ACCOUNT_SUFFIX: + if owner_email.endswith(suffix): + return results try: authors_content = '' |