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

Issue 1759553007: Move depot_tools presubmit trybot from chromium.linux to infra. (Closed)

Created:
4 years, 9 months ago by tandrii(chromium)
Modified:
4 years, 9 months ago
Reviewers:
phajdan.jr, Michael Achenbach, Paweł Hajdan Jr.
CC:
chromium-reviews, infra-reviews+build_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Move depot_tools presubmit trybot from chromium.linux to infra. There is no longer reason to keep it on tryserver.chromium.linux, as it's just occupying one dedicated bot. After move, it'll share 5 bots + we have 1 extra bot on tryserver.infra :) Corresponding depot_tools CL: https://codereview.chromium.org/1763013002, land OR revert both Cls in tandem. R=phajdan.jr@chromiu.org,machenbach@chromium.org NOPRESUBMIT=true BUG= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299099

Patch Set 1 #

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

Messages

Total messages: 16 (9 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1759553007/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1759553007/1
4 years, 9 months ago (2016-03-04 12:44:14 UTC) #2
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 9 months ago (2016-03-04 12:59:45 UTC) #4
Paweł Hajdan Jr.
LGTM Of course this'll require a corresponding CL to switch current users of this bot ...
4 years, 9 months ago (2016-03-04 13:26:12 UTC) #6
tandrii(chromium)
On 2016/03/04 13:26:12, Paweł Hajdan Jr. wrote: > LGTM > > Of course this'll require ...
4 years, 9 months ago (2016-03-04 13:35:02 UTC) #7
tandrii(chromium)
and a master restart scheduled in 15 minutes https://chromereviews.googleplex.com/373697013.
4 years, 9 months ago (2016-03-04 13:36:41 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1759553007/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1759553007/1
4 years, 9 months ago (2016-03-04 13:50:43 UTC) #14
commit-bot: I haz the power
4 years, 9 months ago (2016-03-04 13:51:05 UTC) #16
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299099

Powered by Google App Engine
This is Rietveld 408576698