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

Issue 2199003002: Enable auto reboot of Windows Clang builders (Closed)

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

Description

Enable auto reboot of Windows Clang builders We've seen recent problems with bots running out of memory recently, see e.g. bugs 633273 and 633288. BUG=none Committed: https://chromium.googlesource.com/chromium/tools/build/+/68ab4b8eb098f444cc9277badcd00b02e5df120f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -20 lines) Patch
M masters/master.chromium.fyi/master.cfg View 19 chunks +20 lines, -20 lines 0 comments Download

Messages

Total messages: 17 (6 generated)
hans
Please take a look.
4 years, 4 months ago (2016-08-01 20:05:17 UTC) #2
Nico
lgtm Thanks for doing this!
4 years, 4 months ago (2016-08-01 20:07:23 UTC) #3
hans
+rnk fyi
4 years, 4 months ago (2016-08-01 20:08:04 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2199003002/1
4 years, 4 months ago (2016-08-01 20:08:15 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: Recipe Roll Downstream Tester on luci.infra.try (JOB_FAILED, https://luci-milo.appspot.com/swarming/task/30608d0612512710)
4 years, 4 months ago (2016-08-01 20:14:25 UTC) #8
hans
"Recipe Roll Downstream Tester" failed for unknown reasons agable: does that mean anything to you, ...
4 years, 4 months ago (2016-08-01 20:18:50 UTC) #9
agable
-me +martiniss
4 years, 4 months ago (2016-08-01 20:24:22 UTC) #11
martiniss
On 2016/08/01 at 20:24:22, agable wrote: > -me +martiniss Oh. Sorry about that, the "failed ...
4 years, 4 months ago (2016-08-01 20:33:19 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2199003002/1
4 years, 4 months ago (2016-08-01 20:33:38 UTC) #14
martiniss
On 2016/08/01 at 20:33:38, commit-bot wrote: > CQ is trying da patch. Follow status at ...
4 years, 4 months ago (2016-08-01 20:34:41 UTC) #15
commit-bot: I haz the power
4 years, 4 months ago (2016-08-01 20:40:13 UTC) #17
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/68ab4b8eb098f444cc92...

Powered by Google App Engine
This is Rietveld 408576698