Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(367)

Issue 1597553003: Moving mac_chromium_asan_rel_ng to cq_builders for better resource allocation (Closed)

Created:
4 years, 11 months ago by lijeffrey
Modified:
4 years, 11 months ago
Reviewers:
Sergey Berezin
CC:
chromium-reviews, infra-reviews+build_chromium.org, iannucci, Paweł Hajdan Jr.
Base URL:
https://chromium.googlesource.com/chromium/tools/build@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Moving mac_chromium_asan_rel_ng from optional1_builders to cq_builders for better resource allocation Since optional1_slaves only has 2 bots, it is possible for the estimated resource capacity to reach 6 based on the past month's load. cq_builders has capacity of around 90 bots and seldom goes above 60, so requests to mac_chromium_asan_rel_ng can be handled by bots in cq_builders instead to prevent jobs from piling up in the pending queue. BUG= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=298291

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M masters/master.tryserver.chromium.mac/slaves.cfg View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
lijeffrey
Hey Sergey, thanks for the info about buildbot and showing me how to navigate the ...
4 years, 11 months ago (2016-01-15 23:12:07 UTC) #3
Sergey Berezin
LGTM, thanks! +phajdan.jr FYI
4 years, 11 months ago (2016-01-15 23:18:49 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1597553003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1597553003/1
4 years, 11 months ago (2016-01-15 23:26:03 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1) as http://src.chromium.org/viewvc/chrome?view=rev&revision=298291
4 years, 11 months ago (2016-01-15 23:28:36 UTC) #8
Paweł Hajdan Jr.
4 years, 11 months ago (2016-01-21 11:21:11 UTC) #9
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1614843002/ by phajdan.jr@chromium.org.

The reason for reverting is: We're now seeing pending queues for CQ builders in
this pool.

mac_chromium_asan_rel_ng is not a CQ builder, so moving it back to optional
pool. I'm also removing it from a CQ experiment, see
https://codereview.chromium.org/1611163002 .

If you'd like to request more resources, please do so explicitly..

Powered by Google App Engine
This is Rietveld 408576698