Skip to content

Commit

Permalink
Remove usage of EnsureCQIncludeTrybotsAreAdded
Browse files Browse the repository at this point in the history
We will use path_regexes in the future in cq.cfg if required.

NoTry: true
Bug: skia:8542
Change-Id: Icdba5d3beee6e8b58a9b070b4db8c66cc8f71ac0
Reviewed-on: https://skia-review.googlesource.com/c/170355
Commit-Queue: Ravi Mistry <[email protected]>
Reviewed-by: Eric Boren <[email protected]>
  • Loading branch information
rmistry authored and Skia Commit-Bot committed Nov 12, 2018
1 parent 96a1709 commit 6b90dfe
Showing 1 changed file with 0 additions and 32 deletions.
32 changes: 0 additions & 32 deletions PRESUBMIT.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,21 +40,6 @@
DOCS_PREVIEW_URL = 'https://skia.org/?cl='
GOLD_TRYBOT_URL = 'https://gold.skia.org/search?issue='

# Path to CQ bots feature is described in https://bug.skia.org/4364
PATH_PREFIX_TO_EXTRA_TRYBOTS = {
'src/opts/': ('skia.primary:'
'Test-Debian9-Clang-GCE-CPU-AVX2-x86_64-Release-All-SKNX_NO_SIMD'),
'include/private/SkAtomics.h': ('skia.primary:'
'Test-Debian9-Clang-GCE-CPU-AVX2-x86_64-Release-All-TSAN,'
'Test-Ubuntu17-Clang-Golo-GPU-QuadroP400-x86_64-Release-All-TSAN'
),

# Below are examples to show what is possible with this feature.
# 'src/svg/': 'master1:abc;master2:def',
# 'src/svg/parser/': 'master3:ghi,jkl;master4:mno',
# 'src/image/SkImage_Base.h': 'master5:pqr,stu;master1:abc1;master2:def',
}

SERVICE_ACCOUNT_SUFFIX = [
'@%s.iam.gserviceaccount.com' % project for project in [
'skia-buildbots.google.com', 'skia-swarming-bots', 'skia-public',
Expand Down Expand Up @@ -493,7 +478,6 @@ def PostUploadHook(cl, change, output_api):
work on them.
* Adds 'No-Presubmit: true' for non master branch changes since those don't
run the presubmit checks.
* Adds extra trybots for the paths defined in PATH_TO_EXTRA_TRYBOTS.
"""

results = []
Expand Down Expand Up @@ -567,22 +551,6 @@ def PostUploadHook(cl, change, output_api):
output_api.PresubmitNotifyResult(
'Branch changes do not run the presubmit checks.'))

# Automatically set Cq-Include-Trybots if any of the changed files here
# begin with the paths of interest.
bots_to_include = []
for affected_file in change.AffectedFiles():
affected_file_path = affected_file.LocalPath()
for path_prefix, extra_bots in PATH_PREFIX_TO_EXTRA_TRYBOTS.iteritems():
if affected_file_path.startswith(path_prefix):
results.append(
output_api.PresubmitNotifyResult(
'Your CL modifies the path %s.\nAutomatically adding %s to '
'the CL description.' % (affected_file_path, extra_bots)))
bots_to_include.append(extra_bots)
if bots_to_include:
output_api.EnsureCQIncludeTrybotsAreAdded(
cl, bots_to_include, new_description_lines)

# If the description has changed update it.
if new_description_lines != original_description_lines:
# Add a new line separating the new contents from the old contents.
Expand Down

0 comments on commit 6b90dfe

Please sign in to comment.