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

Issue 1992083002: Add 30 more slaves into 'LTO Linux Perf' buildbot. (Closed)

Created:
4 years, 7 months ago by krasin
Modified:
4 years, 7 months ago
Reviewers:
johnw1, shinyak
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Add 30 more slaves into 'LTO Linux Perf' buildbot. The goal is to verify that we can (on average) have a build for every Chromium CL. If that's true, these slaves will be moved to the real Perf Linux Builder: https://build.chromium.org/p/chromium.perf/builders/Linux%20Builder and then LTO will be launched on Linux. BUG=453195, 612214 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300641

Patch Set 1 #

Patch Set 2 : fix typo #

Patch Set 3 : fix presubmit warning #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -2 lines) Patch
M masters/master.chromium.fyi/slaves.cfg View 1 2 2 chunks +10 lines, -1 line 0 comments Download
M scripts/slave/compile.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 12 (3 generated)
krasin
4 years, 7 months ago (2016-05-18 22:34:03 UTC) #2
shinyak
lgtm
4 years, 7 months ago (2016-05-18 23:27:36 UTC) #3
krasin
Thank you, Shinya! John, can you please approve the changes in masters/master.chromium.fyi/slaves.cfg?
4 years, 7 months ago (2016-05-18 23:36:46 UTC) #4
johnw1
lgtm
4 years, 7 months ago (2016-05-18 23:38:42 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1992083002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1992083002/40001
4 years, 7 months ago (2016-05-18 23:49:29 UTC) #7
krasin
Thanks! Does this change require a master restart to be effective?
4 years, 7 months ago (2016-05-18 23:49:53 UTC) #8
commit-bot: I haz the power
Committed patchset #3 (id:40001) as http://src.chromium.org/viewvc/chrome?view=rev&revision=300641
4 years, 7 months ago (2016-05-18 23:53:07 UTC) #10
johnw1
On 2016/05/18 23:49:53, krasin wrote: > Thanks! Does this change require a master restart to ...
4 years, 7 months ago (2016-05-18 23:53:20 UTC) #11
krasin
4 years, 7 months ago (2016-05-18 23:56:18 UTC) #12
Message was sent while issue was closed.
Thanks, restart requested: https://crbug.com/613009

Powered by Google App Engine
This is Rietveld 408576698