Add a try builder to run boringssl presubmits We now have a few more PRESUBMIT.py checks in the codebase. Not everyone runs these before uploading (e.g. folks who don't use depot_tools), so add coverage on the CQ itself. Adds this new builder as includable_only, since it might take a bit of troubleshooting to get it working. It will be enabled by default once it's working. Change-Id: Ide54a983b9abf280dd1a8e1be15839336a6a6964 Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/84809 Auto-Submit: Lily Chen <chlily@google.com> Reviewed-by: David Benjamin <davidben@google.com> Commit-Queue: David Benjamin <davidben@google.com>
diff --git a/infra/config/generated/commit-queue.cfg b/infra/config/generated/commit-queue.cfg index c8b8361..e52b2ad 100644 --- a/infra/config/generated/commit-queue.cfg +++ b/infra/config/generated/commit-queue.cfg
@@ -194,6 +194,10 @@ name: "boringssl/try/mac_small" } builders { + name: "boringssl/try/presubmit" + includable_only: true + } + builders { name: "boringssl/try/win32" includable_only: true }
diff --git a/infra/config/generated/cr-buildbucket.cfg b/infra/config/generated/cr-buildbucket.cfg index 39e2875..ec93e53 100644 --- a/infra/config/generated/cr-buildbucket.cfg +++ b/infra/config/generated/cr-buildbucket.cfg
@@ -2247,6 +2247,21 @@ service_account: "boringssl-try-builder@chops-service-accounts.iam.gserviceaccount.com" } builders { + name: "presubmit" + swarming_host: "chromium-swarm.appspot.com" + dimensions: "cpu:x86-64" + dimensions: "os:Ubuntu-24.04" + dimensions: "pool:luci.flex.try" + recipe { + name: "presubmit" + cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" + cipd_version: "refs/heads/main" + properties_j: "repo_name:\"boringssl\"" + } + execution_timeout_secs: 1800 + service_account: "boringssl-try-builder@chops-service-accounts.iam.gserviceaccount.com" + } + builders { name: "win32" swarming_host: "chromium-swarm.appspot.com" dimensions: "cpu:x86-64"
diff --git a/infra/config/main.star b/infra/config/main.star index dccedd4..e24c102 100755 --- a/infra/config/main.star +++ b/infra/config/main.star
@@ -346,6 +346,17 @@ # properties rather than parsing names. Then we can add new configurations # without having to touch multiple repositories. +cq_builder( + "presubmit", + LINUX_HOST, + recipe = "presubmit", + # TODO(chlily): Enable when ready. + cq_enabled = False, + properties = { + "repo_name": "boringssl", + }, +) + both_builders( "android_aarch64", WALLEYE_HOST,