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

Issue 1917883003: This moves all of the remaining 32-bit linux builders onto 64-bit hosts. (Closed)

Created:
4 years, 8 months ago by Dirk Pranke
Modified:
4 years, 8 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews, infra-reviews+build_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

This moves all of the remaining 32-bit linux builders onto 64-bit machines. The last phase of cleaning up the Linux 32 configs is to replace the builders and testers with normal 64-bit machines; we no longer wish to support the weird '64-bit kernel with 32-bit userspace' configs that we had before. The waterfall bots are repurposed valgrind machines. The tryservers are moved into the existing optional pool of machines, since they won't be used often. R=thestig@chromium.org BUG=600903, 600904, 590004 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300213

Patch Set 1 : nits #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -43 lines) Patch
M masters/master.chromium.fyi/slaves.cfg View 1 chunk +2 lines, -2 lines 1 comment Download
M masters/master.chromium.linux/slaves.cfg View 2 chunks +4 lines, -4 lines 0 comments Download
M masters/master.tryserver.chromium.linux/slaves.cfg View 4 chunks +5 lines, -37 lines 0 comments Download

Messages

Total messages: 13 (5 generated)
Dirk Pranke
4 years, 8 months ago (2016-04-26 00:42:24 UTC) #3
Lei Zhang
https://codereview.chromium.org/1917883003/diff/2/masters/master.chromium.fyi/slaves.cfg File masters/master.chromium.fyi/slaves.cfg (right): https://codereview.chromium.org/1917883003/diff/2/masters/master.chromium.fyi/slaves.cfg#newcode208 masters/master.chromium.fyi/slaves.cfg:208: 'hostname': 'vm925-m1', So this is a 64-bit kernel+userland machine, ...
4 years, 8 months ago (2016-04-26 01:18:24 UTC) #4
Dirk Pranke
On 2016/04/26 01:18:24, Lei Zhang wrote: > https://codereview.chromium.org/1917883003/diff/2/masters/master.chromium.fyi/slaves.cfg > File masters/master.chromium.fyi/slaves.cfg (right): > > https://codereview.chromium.org/1917883003/diff/2/masters/master.chromium.fyi/slaves.cfg#newcode208 ...
4 years, 8 months ago (2016-04-26 01:40:25 UTC) #5
Lei Zhang
On 2016/04/26 01:40:25, Dirk Pranke wrote: > Correct. Or, if it can't, I'll fix it ...
4 years, 8 months ago (2016-04-26 02:27:14 UTC) #7
Dirk Pranke
On 2016/04/26 02:27:14, Lei Zhang wrote: > On 2016/04/26 01:40:25, Dirk Pranke wrote: > > ...
4 years, 8 months ago (2016-04-26 02:34:41 UTC) #8
Lei Zhang
lgtm
4 years, 8 months ago (2016-04-26 03:48:15 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1917883003/2 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1917883003/2
4 years, 8 months ago (2016-04-26 21:54:50 UTC) #11
commit-bot: I haz the power
4 years, 8 months ago (2016-04-26 22:21:49 UTC) #13
Message was sent while issue was closed.
Committed patchset #1 (id:2) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300213

Powered by Google App Engine
This is Rietveld 408576698