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

Issue 2066013003: Moving Findit variable builders to new high-end cpu slave (Closed)

Created:
4 years, 6 months ago by lijeffrey
Modified:
4 years, 6 months ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, stgao, chanli
Base URL:
https://chromium.googlesource.com/chromium/tools/build@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Moving Findit variable builders to new high-end cpu slave BUG=620207 Committed: https://chromium.googlesource.com/chromium/tools/build/+/aeb18b9f5f758b6b8aefe411cb84f6819ab50930

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressing comments #

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

Messages

Total messages: 12 (6 generated)
lijeffrey
ptal Thanks Ryan for bot 799! This change moves one of the variable builders (with ...
4 years, 6 months ago (2016-06-15 05:55:16 UTC) #4
stgao
https://codereview.chromium.org/2066013003/diff/1/masters/master.tryserver.chromium.linux/slaves.cfg File masters/master.tryserver.chromium.linux/slaves.cfg (right): https://codereview.chromium.org/2066013003/diff/1/masters/master.tryserver.chromium.linux/slaves.cfg#newcode101 masters/master.tryserver.chromium.linux/slaves.cfg:101: optional_highcpu_slaves = [799] I guess we want to dedicate ...
4 years, 6 months ago (2016-06-15 06:12:39 UTC) #6
lijeffrey
https://codereview.chromium.org/2066013003/diff/1/masters/master.tryserver.chromium.linux/slaves.cfg File masters/master.tryserver.chromium.linux/slaves.cfg (right): https://codereview.chromium.org/2066013003/diff/1/masters/master.tryserver.chromium.linux/slaves.cfg#newcode101 masters/master.tryserver.chromium.linux/slaves.cfg:101: optional_highcpu_slaves = [799] On 2016/06/15 06:12:39, stgao wrote: > ...
4 years, 6 months ago (2016-06-15 18:10:40 UTC) #7
Sergey Berezin
lgtm
4 years, 6 months ago (2016-06-15 19:15:15 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2066013003/20001
4 years, 6 months ago (2016-06-15 20:40:29 UTC) #10
commit-bot: I haz the power
4 years, 6 months ago (2016-06-15 20:43:46 UTC) #12
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/tools/build/+/aeb18b9f5f758b6b8aef...

Powered by Google App Engine
This is Rietveld 408576698