Revert "Temporarily set android_browsertests as ci_only on Android arm64 CQ" [chromium/src : main]
0 views
Skip to first unread message
Haiyang Pan (Gerrit)
Jan 13, 2026, 12:13:01 PMJan 13
to Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Haiyang Pan has uploaded the change for review![]( "Open in Gerrit")
Haiyang Pan would like Ben Pastene and Chromium LUCI CQ to review this change.
Commit message
Revert "Temporarily set android_browsertests as ci_only on Android arm64 CQ"
This reverts commit 42e1ed60888c8dfc048f4d31bb216131aa5332bb.
Reason for revert: 86 more devices are added to the pool which
expands the capacity from 100ish to 190ish. Hopefully this is
sufficient
Original change's description:
> Temporarily set android_browsertests as ci_only on Android arm64 CQ
>
> As the physical devices cannot consume the test requests right now.
>
> No-Try: true
> Bug: 475299611
> Change-Id: I0676f20b4565a051bb57763a80b17190d9dd8d13
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7457800
> Auto-Submit: Haiyang Pan <hy...@google.com>
> Reviewed-by: Ben Pastene <bpas...@chromium.org>
> Commit-Queue: Haiyang Pan <hy...@google.com>
> Commit-Queue: Ben Pastene <bpas...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1568112}
Bug: 475299611
Change-Id: I352c1949ad1b5b1758af5df007decf81c380481f
Change diff
Change information
Files:
- M infra/config/generated/builders/ci/android-14-arm64-rel/targets/chromium.android.json
- M infra/config/generated/builders/try/android-14-arm64-rel/targets/chromium.android.json
- M infra/config/generated/builders/try/android-arm64-rel/targets/chromium.android.json
- M infra/config/generated/cq-tests.md
- M infra/config/subprojects/chromium/ci/chromium.android.star
Change size: XS
Delta: 5 files changed, 1 insertion(+), 6 deletions(-)
Related details
Attention set is empty
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Haiyang Pan (Gerrit)
Jan 13, 2026, 1:45:06 PMJan 13
to Rubber Stamper, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Ben Pastene
New activity on the change![]( "Open in Gerrit")
Related details
Attention is currently required from:
- Ben Pastene
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rubber Stamper (Gerrit)
Jan 13, 2026, 1:45:18 PMJan 13
to Haiyang Pan, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Ben Pastene
Rubber Stamper voted Bot-Commit+1![]( "Open in Gerrit")
| Bot-Commit | +1 |
Related details
Attention is currently required from:
- Ben Pastene
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ben Pastene (Gerrit)
Jan 13, 2026, 2:34:43 PMJan 13
to Haiyang Pan, Rubber Stamper, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Haiyang Pan
Ben Pastene added 1 comment![]( "Open in Gerrit")
Attention is currently required from:
- Haiyang Pan
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Haiyang Pan (Gerrit)
Jan 13, 2026, 2:50:05 PMJan 13
to Rubber Stamper, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Ben Pastene
Haiyang Pan added 1 comment![]( "Open in Gerrit")
Commit Message
Line 13, Patchset 1 (Latest):
sufficientBen Pastene . resolved
What caused the expirations yesterday? http://screen/5DvoLypHzYgNUdt
Haiyang Pan
My theory is that due to https://crbug.com/475532838, some shards in android_browsertests can take a long time 2x, or even 4x in the worst case. This ate the remaining testing capacity buffer, and causes the pending time.
Related details
Attention is currently required from:
- Ben Pastene
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ben Pastene (Gerrit)
Jan 13, 2026, 4:06:29 PMJan 13
to Haiyang Pan, Rubber Stamper, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Haiyang Pan
Ben Pastene added 1 comment![]( "Open in Gerrit")
Commit Message
Line 13, Patchset 1 (Latest):
sufficientBen Pastene . resolved
Haiyang PanWhat caused the expirations yesterday? http://screen/5DvoLypHzYgNUdt
My theory is that due to https://crbug.com/475532838, some shards in android_browsertests can take a long time 2x, or even 4x in the worst case. This ate the remaining testing capacity buffer, and causes the pending time.
Ben Pastene
Has that been addressed? If not, is it safe to re-enable the suite in the CQ?
Related details
Attention is currently required from:
- Haiyang Pan
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Haiyang Pan (Gerrit)
Jan 13, 2026, 4:28:19 PMJan 13
to Rubber Stamper, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Ben Pastene
Haiyang Pan added 1 comment![]( "Open in Gerrit")
Commit Message
Line 13, Patchset 1 (Latest):
sufficientBen Pastene . resolved
Haiyang PanWhat caused the expirations yesterday? http://screen/5DvoLypHzYgNUdt
Ben PasteneMy theory is that due to https://crbug.com/475532838, some shards in android_browsertests can take a long time 2x, or even 4x in the worst case. This ate the remaining testing capacity buffer, and causes the pending time.
Has that been addressed? If not, is it safe to re-enable the suite in the CQ?
Haiyang Pan
The issue is not addressed yet. I just check the build cycle time and found it regresses quite a bit since Jan 7: http://shortn/_Wz1PY16PIH. Let's wait a bit then.
Related details
Attention is currently required from:
- Ben Pastene
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Haiyang Pan (Gerrit)
Jan 15, 2026, 5:20:13 PMJan 15
to Rubber Stamper, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Ben Pastene
Haiyang Pan voted and added 1 comment![]( "Open in Gerrit")
Votes added by Haiyang Pan
| Commit-Queue | +1 |
1 comment
Patchset-level comments
File-level comment, Patchset 1 (Latest):
Haiyang Pan . resolved
The Pixel 7 pool is in a much better state now: https://crbug.com/475299611#comment7
Plus the per-shard runtime is 10ish now which is also good.
So add the suite back to CQ.
Related details
Attention is currently required from:
- Ben Pastene
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ben Pastene (Gerrit)
Jan 15, 2026, 5:44:07 PMJan 15
to Haiyang Pan, Rubber Stamper, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Haiyang Pan
Ben Pastene voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Haiyang Pan
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rubber Stamper (Gerrit)
Jan 15, 2026, 5:49:20 PMJan 15
to Haiyang Pan, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Haiyang Pan
Rubber Stamper voted Bot-Commit+1![]( "Open in Gerrit")
| Bot-Commit | +1 |
Related details
Attention is currently required from:
- Haiyang Pan
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Haiyang Pan (Gerrit)
Jan 15, 2026, 5:51:30 PMJan 15
to Rubber Stamper, Ben Pastene, Chromium LUCI CQ, chromium...@chromium.org
Haiyang Pan voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Jan 15, 2026, 6:14:33 PMJan 15
to Haiyang Pan, Rubber Stamper, Ben Pastene, chromium...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Unreviewed changes
1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
Change information
Commit message:
Revert "Temporarily set android_browsertests as ci_only on Android arm64 CQ"
This reverts commit 42e1ed60888c8dfc048f4d31bb216131aa5332bb.
Reason for revert: The pool size is restored from 200ish to 450+ so
should be good to process the test requests.
Original change's description:
> Temporarily set android_browsertests as ci_only on Android arm64 CQ
>
> As the physical devices cannot consume the test requests right now.
>
> No-Try: true
> Bug: 475299611
> Change-Id: I0676f20b4565a051bb57763a80b17190d9dd8d13
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7457800
> Auto-Submit: Haiyang Pan <hy...@google.com>
> Reviewed-by: Ben Pastene <bpas...@chromium.org>
> Commit-Queue: Haiyang Pan <hy...@google.com>
> Commit-Queue: Ben Pastene <bpas...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1568112}
Bug: 475299611
Change-Id: I352c1949ad1b5b1758af5df007decf81c380481f
Commit-Queue: Haiyang Pan <hy...@google.com>
Reviewed-by: Ben Pastene <bpas...@chromium.org>
Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1570042}
Files:
- M infra/config/generated/builders/ci/android-14-arm64-rel/targets/chromium.android.json
- M infra/config/generated/builders/try/android-14-arm64-rel/targets/chromium.android.json
- M infra/config/generated/builders/try/android-arm64-rel/targets/chromium.android.json
- M infra/config/generated/cq-tests.md
- M infra/config/subprojects/chromium/ci/chromium.android.star
Change size: XS
Delta: 5 files changed, 1 insertion(+), 6 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Ben Pastene, Bot-Commit+1 by Rubber Stamper
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages