Skip to content

Commit 72e0b5b

Browse files
bpasteneCommit Bot
bpastene
authored and
Commit Bot
committed
infra: Remove all CQ experiments while the CQ is overloaded.
A small attempt at reducing overall traffic to various pools. Bug: 1116532 Change-Id: I9c36e08a9babfa4a69d62bb246950ae48890e94c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2357412 Reviewed-by: Erik Staab <estaab@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#798233}
1 parent af0831d commit 72e0b5b

File tree

5 files changed

+39
-77
lines changed

5 files changed

+39
-77
lines changed

‎infra/config/generated/commit-queue.cfg

+6-28
Original file line numberDiff line numberDiff line change
@@ -215,10 +215,7 @@ config_groups {
215215
}
216216
builders {
217217
name: "chromium/try/android-marshmallow-x86-rel"
218-
experiment_percentage: 5
219-
location_regexp: ".*"
220-
location_regexp_exclude: ".+/[+]/docs/.+"
221-
location_regexp_exclude: ".+/[+]/infra/config/.+"
218+
includable_only: true
222219
}
223220
builders {
224221
name: "chromium/try/android-nougat-arm64-rel"
@@ -550,10 +547,7 @@ config_groups {
550547
}
551548
builders {
552549
name: "chromium/try/fuchsia-compile-x64-dbg"
553-
experiment_percentage: 50
554-
location_regexp: ".*"
555-
location_regexp_exclude: ".+/[+]/docs/.+"
556-
location_regexp_exclude: ".+/[+]/infra/config/.+"
550+
includable_only: true
557551
}
558552
builders {
559553
name: "chromium/try/fuchsia-fyi-arm64-dbg"
@@ -861,10 +855,7 @@ config_groups {
861855
}
862856
builders {
863857
name: "chromium/try/ios-simulator-code-coverage"
864-
experiment_percentage: 3
865-
location_regexp: ".*"
866-
location_regexp_exclude: ".+/[+]/docs/.+"
867-
location_regexp_exclude: ".+/[+]/infra/config/.+"
858+
includable_only: true
868859
}
869860
builders {
870861
name: "chromium/try/ios-simulator-cr-recipe"
@@ -1035,14 +1026,7 @@ config_groups {
10351026
}
10361027
builders {
10371028
name: "chromium/try/linux-perfetto-rel"
1038-
experiment_percentage: 100
1039-
location_regexp: ".+/[+]/base/trace_event/.+"
1040-
location_regexp: ".+/[+]/base/tracing/.+"
1041-
location_regexp: ".+/[+]/components/tracing/.+"
1042-
location_regexp: ".+/[+]/content/browser/tracing/.+"
1043-
location_regexp: ".+/[+]/services/tracing/.+"
1044-
location_regexp_exclude: ".+/[+]/docs/.+"
1045-
location_regexp_exclude: ".+/[+]/infra/config/.+"
1029+
includable_only: true
10461030
}
10471031
builders {
10481032
name: "chromium/try/linux-rel"
@@ -1243,10 +1227,7 @@ config_groups {
12431227
}
12441228
builders {
12451229
name: "chromium/try/mac-coverage-rel"
1246-
experiment_percentage: 3
1247-
location_regexp: ".*"
1248-
location_regexp_exclude: ".+/[+]/docs/.+"
1249-
location_regexp_exclude: ".+/[+]/infra/config/.+"
1230+
includable_only: true
12501231
}
12511232
builders {
12521233
name: "chromium/try/mac-dawn-rel"
@@ -2300,10 +2281,7 @@ config_groups {
23002281
}
23012282
builders {
23022283
name: "chromium/try-m85/fuchsia_arm64"
2303-
experiment_percentage: 50
2304-
location_regexp: ".*"
2305-
location_regexp_exclude: ".+/[+]/docs/.+"
2306-
location_regexp_exclude: ".+/[+]/infra/config/.+"
2284+
includable_only: true
23072285
}
23082286
builders {
23092287
name: "chromium/try-m85/fuchsia_x64"

‎infra/config/generated/cq-builders.md

-28
Original file line numberDiff line numberDiff line change
@@ -337,31 +337,3 @@ builder.
337337
* [`//tools/clang/scripts/update.py`](https://cs.chromium.org/search?q=package:%5Echromium$+file:tools/clang/scripts/update.py)
338338
* [`//ui/gl/.+`](https://cs.chromium.org/chromium/src/ui/gl/)
339339

340-
341-
## Experimental builders
342-
These builders are run on some percentage of builds. Their results are ignored
343-
by CQ. These are often used to test new configurations before they are added
344-
as required builders.
345-
346-
* [android-marshmallow-x86-rel](https://ci.chromium.org/p/chromium/builders/try/android-marshmallow-x86-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+android-marshmallow-x86-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+android-marshmallow-x86-rel))
347-
* Experiment percentage: 5
348-
349-
* [fuchsia-compile-x64-dbg](https://ci.chromium.org/p/chromium/builders/try/fuchsia-compile-x64-dbg) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+fuchsia-compile-x64-dbg)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+fuchsia-compile-x64-dbg))
350-
* Experiment percentage: 50
351-
352-
* [ios-simulator-code-coverage](https://ci.chromium.org/p/chromium/builders/try/ios-simulator-code-coverage) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+ios-simulator-code-coverage)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+ios-simulator-code-coverage))
353-
* Experiment percentage: 3
354-
355-
* [linux-perfetto-rel](https://ci.chromium.org/p/chromium/builders/try/linux-perfetto-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+linux-perfetto-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+linux-perfetto-rel))
356-
* Experiment percentage: 100
357-
358-
Path regular expressions:
359-
* [`//base/trace_event/.+`](https://cs.chromium.org/chromium/src/base/trace_event/)
360-
* [`//base/tracing/.+`](https://cs.chromium.org/chromium/src/base/tracing/)
361-
* [`//components/tracing/.+`](https://cs.chromium.org/chromium/src/components/tracing/)
362-
* [`//content/browser/tracing/.+`](https://cs.chromium.org/chromium/src/content/browser/tracing/)
363-
* [`//services/tracing/.+`](https://cs.chromium.org/chromium/src/services/tracing/)
364-
365-
* [mac-coverage-rel](https://ci.chromium.org/p/chromium/builders/try/mac-coverage-rel) ([definition](https://cs.chromium.org/search?q=package:%5Echromium$+file:/cq.star$+-file:/beta/+-file:/stable/+mac-coverage-rel)) ([matching builders](https://cs.chromium.org/search?q=+file:trybots.py+mac-coverage-rel))
366-
* Experiment percentage: 3
367-

‎infra/config/subprojects/chromium/master-only/try.star

+23-15
Original file line numberDiff line numberDiff line change
@@ -318,9 +318,11 @@ try_.chromium_linux_builder(
318318

319319
try_.chromium_linux_builder(
320320
name = "fuchsia-compile-x64-dbg",
321-
tryjob = try_.job(
322-
experiment_percentage = 50,
323-
),
321+
# TODO(crbug.com/1116532): Move back to experimental once CQ has settled
322+
# down.
323+
#tryjob = try_.job(
324+
# experiment_percentage = 50,
325+
#),
324326
)
325327

326328
try_.chromium_linux_builder(
@@ -386,16 +388,18 @@ try_.chromium_linux_builder(
386388

387389
try_.chromium_linux_builder(
388390
name = "linux-perfetto-rel",
389-
tryjob = try_.job(
390-
experiment_percentage = 100,
391-
location_regexp = [
392-
".+/[+]/base/trace_event/.+",
393-
".+/[+]/base/tracing/.+",
394-
".+/[+]/components/tracing/.+",
395-
".+/[+]/content/browser/tracing/.+",
396-
".+/[+]/services/tracing/.+",
397-
],
398-
),
391+
# TODO(crbug.com/1116532): Move back to experimental once CQ has settled
392+
# down.
393+
#tryjob = try_.job(
394+
# experiment_percentage = 100,
395+
# location_regexp = [
396+
# ".+/[+]/base/trace_event/.+",
397+
# ".+/[+]/base/tracing/.+",
398+
# ".+/[+]/components/tracing/.+",
399+
# ".+/[+]/content/browser/tracing/.+",
400+
# ".+/[+]/services/tracing/.+",
401+
# ],
402+
#),
399403
)
400404

401405
try_.chromium_linux_builder(
@@ -508,7 +512,9 @@ try_.chromium_mac_builder(
508512
name = "mac-coverage-rel",
509513
use_clang_coverage = True,
510514
goma_jobs = goma.jobs.J150,
511-
tryjob = try_.job(experiment_percentage = 3),
515+
# TODO(crbug.com/1116532): Move back to experimental once CQ has settled
516+
# down.
517+
#tryjob = try_.job(experiment_percentage = 3),
512518
)
513519

514520
try_.chromium_mac_builder(
@@ -590,7 +596,9 @@ try_.chromium_mac_ios_builder(
590596
properties = {
591597
"xcode_build_version": "11e146",
592598
},
593-
tryjob = try_.job(experiment_percentage = 3),
599+
# TODO(crbug.com/1116532): Move back to experimental once CQ has settled
600+
# down.
601+
#tryjob = try_.job(experiment_percentage = 3),
594602
)
595603

596604
try_.chromium_mac_ios_builder(

‎infra/config/subprojects/chromium/try.star

+5-3
Original file line numberDiff line numberDiff line change
@@ -127,9 +127,11 @@ try_.chromium_android_builder(
127127
try_.chromium_android_builder(
128128
name = "android-marshmallow-x86-rel",
129129
goma_jobs = goma.jobs.J150,
130-
tryjob = try_.job(
131-
experiment_percentage = 5,
132-
),
130+
# TODO(crbug.com/1116532): Move back to experimental once CQ has settled
131+
# down.
132+
#tryjob = try_.job(
133+
# experiment_percentage = 5,
134+
#),
133135
)
134136

135137
try_.chromium_android_builder(

‎infra/config/subprojects/chromium/versioned/m85/buckets/try.star

+5-3
Original file line numberDiff line numberDiff line change
@@ -334,9 +334,11 @@ try_.chromium_linux_builder(
334334
# monitoring stability and capacity. crbug.com/1042511
335335
try_.chromium_linux_builder(
336336
name = "fuchsia_arm64",
337-
tryjob = try_.job(
338-
experiment_percentage = 50,
339-
),
337+
# TODO(crbug.com/1116532): Move back to experimental once CQ has settled
338+
# down.
339+
#tryjob = try_.job(
340+
# experiment_percentage = 50,
341+
#),
340342
)
341343

342344
try_.chromium_linux_builder(

0 commit comments

Comments
 (0)