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

Issue 761473002: Remove obsolete trybots, round #1 (Closed)

Created:
6 years ago by Paweł Hajdan Jr.
Modified:
6 years ago
Reviewers:
Sergiy Byelozyorov
CC:
chromium-reviews, cmp-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Project:
tools
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -293 lines) Patch
M masters/master.tryserver.chromium.linux/master.cfg View 6 chunks +0 lines, -185 lines 0 comments Download
M masters/master.tryserver.chromium.linux/slaves.cfg View 2 chunks +0 lines, -4 lines 0 comments Download
M masters/master.tryserver.chromium.mac/master.cfg View 4 chunks +1 line, -63 lines 0 comments Download
M masters/master.tryserver.chromium.mac/slaves.cfg View 3 chunks +0 lines, -25 lines 2 comments Download
M masters/master.tryserver.chromium.win/master.cfg View 2 chunks +0 lines, -15 lines 0 comments Download
M masters/master.tryserver.chromium.win/slaves.cfg View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 8 (2 generated)
Paweł Hajdan Jr.
6 years ago (2014-11-25 14:41:54 UTC) #2
Sergiy Byelozyorov
lgtm w/ comments https://codereview.chromium.org/761473002/diff/1/masters/master.tryserver.chromium.mac/slaves.cfg File masters/master.tryserver.chromium.mac/slaves.cfg (left): https://codereview.chromium.org/761473002/diff/1/masters/master.tryserver.chromium.mac/slaves.cfg#oldcode164 masters/master.tryserver.chromium.mac/slaves.cfg:164: } for i in range(84, 90) ...
6 years ago (2014-11-25 15:33:44 UTC) #3
Sergiy Byelozyorov
I assume you've followed the discussion and won't remove any bots that are still used. ...
6 years ago (2014-11-25 15:34:47 UTC) #4
Paweł Hajdan Jr.
https://codereview.chromium.org/761473002/diff/1/masters/master.tryserver.chromium.mac/slaves.cfg File masters/master.tryserver.chromium.mac/slaves.cfg (left): https://codereview.chromium.org/761473002/diff/1/masters/master.tryserver.chromium.mac/slaves.cfg#oldcode164 masters/master.tryserver.chromium.mac/slaves.cfg:164: } for i in range(84, 90) On 2014/11/25 15:33:44, ...
6 years ago (2014-11-25 16:16:37 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/761473002/1
6 years ago (2014-11-25 16:17:25 UTC) #7
commit-bot: I haz the power
6 years ago (2014-11-25 16:18:07 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=293103

Powered by Google App Engine
This is Rietveld 408576698