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

Issue 2072823003: master.chromium.infra: Remove redundant tools/build builder (Closed)

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

Description

master.chromium.infra: Remove redundant tools/build builder BUG=614200 Committed: https://chromium.googlesource.com/chromium/tools/build/+/67abf06940be79d423e2af203584ad1c3d64de6e

Patch Set 1 #

Patch Set 2 : master.chromium.infra: fix recipe simulation #

Total comments: 1

Patch Set 3 : Whoops remote_run #

Patch Set 4 : Fix slaves.cfg. #

Patch Set 5 : remove old scheduler #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -18 lines) Patch
M masters/master.chromium.infra/master.cfg View 1 2 3 4 2 chunks +0 lines, -15 lines 0 comments Download
M masters/master.chromium.infra/slaves.cfg View 1 2 3 3 chunks +0 lines, -3 lines 0 comments Download

Messages

Total messages: 19 (10 generated)
martiniss
PTAL (messed up master.cfg at first :( )
4 years, 6 months ago (2016-06-17 03:04:11 UTC) #2
martiniss
https://codereview.chromium.org/2072823003/diff/20001/masters/master.chromium.infra/master.cfg File masters/master.chromium.infra/master.cfg (left): https://codereview.chromium.org/2072823003/diff/20001/masters/master.chromium.infra/master.cfg#oldcode201 masters/master.chromium.infra/master.cfg:201: 'name': 'build-recipes-tests', https://build.chromium.org/p/chromium.tools.build/builders/recipe-simulation_trusty64 for this
4 years, 6 months ago (2016-06-17 03:05:52 UTC) #3
tandrii(chromium)
lgtm
4 years, 6 months ago (2016-06-17 11:58:16 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2072823003/40001
4 years, 6 months ago (2016-06-22 18:33:12 UTC) #7
commit-bot: I haz the power
Try jobs failed on following builders: Build Presubmit on tryserver.infra (JOB_FAILED, https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/builds/5630)
4 years, 6 months ago (2016-06-22 18:40:23 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2072823003/60001
4 years, 6 months ago (2016-06-24 00:12:19 UTC) #12
commit-bot: I haz the power
Try jobs failed on following builders: Build Presubmit on tryserver.infra (JOB_FAILED, https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/builds/5672)
4 years, 6 months ago (2016-06-24 00:19:32 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2072823003/80001
4 years, 6 months ago (2016-06-24 00:46:29 UTC) #17
commit-bot: I haz the power
4 years, 6 months ago (2016-06-24 00:49:42 UTC) #19
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/tools/build/+/67abf06940be79d423e2...

Powered by Google App Engine
This is Rietveld 408576698