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

Issue 1419783008: Swarm telemetry_gpu_unittests on "Windows Tests", "Mac tests", "Linux tests" and FYI bot (Closed)

Created:
5 years, 1 month ago by nednguyen
Modified:
5 years, 1 month ago
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Swarm telemetry_gpu_unittests on "Windows Tests", "Mac tests", "Linux tests" and FYI bot Related patches to watch for with swarming enabled: *Add some test failures to telemetry_gpu_unittests: https://codereview.chromium.org/1414983005 *Break the whole telemetry_gpu_unittests test harness: https://codereview.chromium.org/1422613003 BUG=507796 Committed: https://crrev.com/0a4c2e1c47c6c44d045cc7cf0a1ce6ae21355012 Cr-Commit-Position: refs/heads/master@{#356695}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+72 lines, -18 lines) Patch
M testing/buildbot/chromium.fyi.json View 7 chunks +28 lines, -7 lines 0 comments Download
M testing/buildbot/chromium.linux.json View 2 chunks +8 lines, -2 lines 0 comments Download
M testing/buildbot/chromium.mac.json View 5 chunks +20 lines, -5 lines 0 comments Download
M testing/buildbot/chromium.win.json View 4 chunks +16 lines, -4 lines 0 comments Download

Messages

Total messages: 20 (10 generated)
nednguyen
5 years, 1 month ago (2015-10-28 14:24:45 UTC) #3
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1419783008/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1419783008/1
5 years, 1 month ago (2015-10-28 16:24:44 UTC) #5
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 1 month ago (2015-10-28 16:57:54 UTC) #7
Ken Russell (switch to Gerrit)
First, this is great! The first CL's try jobs look good. Second, it looks like ...
5 years, 1 month ago (2015-10-28 18:32:21 UTC) #9
nednguyen
On 2015/10/28 18:32:21, Ken Russell wrote: > First, this is great! The first CL's try ...
5 years, 1 month ago (2015-10-28 22:43:41 UTC) #14
Ken Russell (switch to Gerrit)
Excellent! Thank you for pushing this through. LGTM
5 years, 1 month ago (2015-10-28 23:51:06 UTC) #15
M-A Ruel
rs lgtm; confirmed telemetry_gpu_unittests on mac was not marked as idempotent; https://chromium-swarm.appspot.com/user/task/2ac8213231ca4410
5 years, 1 month ago (2015-10-28 23:54:22 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1419783008/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1419783008/1
5 years, 1 month ago (2015-10-28 23:56:04 UTC) #18
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 1 month ago (2015-10-29 00:05:41 UTC) #19
commit-bot: I haz the power
5 years, 1 month ago (2015-10-29 00:07:08 UTC) #20
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/0a4c2e1c47c6c44d045cc7cf0a1ce6ae21355012
Cr-Commit-Position: refs/heads/master@{#356695}

Powered by Google App Engine
This is Rietveld 408576698