Commit 2dcf0ac3 authored by Yuke Liao's avatar Yuke Liao Committed by Commit Bot

Reland "[lacros] Enable linux-lacros-rel by default"

This reverts commit 66870c32.

Reason for revert:
1. The root cause is not lacros, symptom is generally seen across all builders.
2. #pending tasks is ~0 now: https://chromium-swarm.appspot.com/task?id=4e21b13cf27db110&o=true&w=true.
3. 400 VMs were temporarily added to the pool to mitigate the outage.

Original change's description:
> Revert "[lacros] Enable linux-lacros-rel by default"
> 
> This reverts commit ac7726b8.
> 
> Reason for revert: reverting to shed load on CQ, crbug.com/1119483
> 
> Original change's description:
> > [lacros] Enable linux-lacros-rel by default
> > 
> > This CL enables linux-lacros-rel on CQ by default.
> > 
> > Bug: 1116180
> > Change-Id: I67bf89a5eefd7382bbd2886f416bc38c411213bf
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360496
> > Reviewed-by: Erik Staab <estaab@chromium.org>
> > Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#799123}
> 
> TBR=estaab@chromium.org,liaoyuke@chromium.org
> 
> # Not skipping CQ checks because original CL landed > 1 day ago.
> 
> Bug: 1116180, 1119483
> Change-Id: I2be0017d3af1e863ec15efaa3c3cdf1ff1764911
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364140
> Reviewed-by: Ben Pastene <bpastene@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#799798}

TBR=estaab@chromium.org,liaoyuke@chromium.org,bpastene@chromium.org

# Not skipping CQ checks because this is a reland.

Bug: 1116180
Bug: 1119483
Change-Id: Ib9211794c4eb92e3e5c721c77f4821dda49d8a0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365223Reviewed-by: default avatarYuke Liao <liaoyuke@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799877}
parent f0d5d6a4
...@@ -997,9 +997,7 @@ config_groups { ...@@ -997,9 +997,7 @@ config_groups {
} }
builders { builders {
name: "chromium/try/linux-lacros-compile-rel" name: "chromium/try/linux-lacros-compile-rel"
location_regexp: ".*" includable_only: true
location_regexp_exclude: ".+/[+]/docs/.+"
location_regexp_exclude: ".+/[+]/infra/config/.+"
} }
builders { builders {
name: "chromium/try/linux-lacros-fyi-rel" name: "chromium/try/linux-lacros-fyi-rel"
...@@ -1007,7 +1005,9 @@ config_groups { ...@@ -1007,7 +1005,9 @@ config_groups {
} }
builders { builders {
name: "chromium/try/linux-lacros-rel" name: "chromium/try/linux-lacros-rel"
includable_only: true location_regexp: ".*"
location_regexp_exclude: ".+/[+]/docs/.+"
location_regexp_exclude: ".+/[+]/infra/config/.+"
} }
builders { builders {
name: "chromium/try/linux-layout-tests-edit-ng" name: "chromium/try/linux-layout-tests-edit-ng"
......
...@@ -49,7 +49,7 @@ These builders must pass before a CL may land that affects files outside of ...@@ -49,7 +49,7 @@ These builders must pass before a CL may land that affects files outside of
* [linux-chromeos-rel](https://ci.chromium.org/p/chromium/builders/try/linux-chromeos-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-chromeos-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-chromeos-rel)) * [linux-chromeos-rel](https://ci.chromium.org/p/chromium/builders/try/linux-chromeos-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-chromeos-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-chromeos-rel))
* [linux-lacros-compile-rel](https://ci.chromium.org/p/chromium/builders/try/linux-lacros-compile-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-lacros-compile-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-lacros-compile-rel)) * [linux-lacros-rel](https://ci.chromium.org/p/chromium/builders/try/linux-lacros-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-lacros-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-lacros-rel))
* [linux-libfuzzer-asan-rel](https://ci.chromium.org/p/chromium/builders/try/linux-libfuzzer-asan-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-libfuzzer-asan-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-libfuzzer-asan-rel)) * [linux-libfuzzer-asan-rel](https://ci.chromium.org/p/chromium/builders/try/linux-libfuzzer-asan-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-libfuzzer-asan-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-libfuzzer-asan-rel))
......
...@@ -271,13 +271,14 @@ try_.chromium_chromiumos_builder( ...@@ -271,13 +271,14 @@ try_.chromium_chromiumos_builder(
main_list_view = "try", main_list_view = "try",
) )
# TODO(crbug.com/1116180): Clean this up once linux-lacros-rel is up.
try_.chromium_chromiumos_builder( try_.chromium_chromiumos_builder(
name = "linux-lacros-compile-rel", name = "linux-lacros-compile-rel",
tryjob = try_.job(),
) )
try_.chromium_chromiumos_builder( try_.chromium_chromiumos_builder(
name = "linux-lacros-rel", name = "linux-lacros-rel",
tryjob = try_.job(),
) )
try_.chromium_chromiumos_builder( try_.chromium_chromiumos_builder(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment