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

Issue 1551183002: Move many tests on the FYI waterfall to swarming. (Closed)

Created:
4 years, 11 months ago by Nico
Modified:
4 years, 3 months 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

Move many tests on the FYI waterfall to swarming. I ran ./testing/buildbot/manage.py --convert foo_test for all tests that were swarmed on some bot in chromium.memory.json. This updated lots of FYI bots, but also some other bots. Also add four blink test binaries to the Mac ASan bot. These binaries run without issues on the ClangToTMacASan bot. BUG=98637

Patch Set 1 #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+963 lines, -5 lines) Patch
M testing/buildbot/chromium.android.json View 9 chunks +27 lines, -0 lines 1 comment Download
M testing/buildbot/chromium.fyi.json View 124 chunks +889 lines, -1 line 2 comments Download
M testing/buildbot/chromium.linux.json View 3 chunks +21 lines, -0 lines 1 comment Download
M testing/buildbot/chromium.memory.json View 2 chunks +24 lines, -0 lines 2 comments Download
M testing/buildbot/manage.py View 2 chunks +2 lines, -4 lines 0 comments Download

Messages

Total messages: 20 (4 generated)
Nico
martiniss to check if the chromium.android.json (and chromium.fyi.json) are ok or if these bots shouldn't ...
4 years, 11 months ago (2015-12-31 16:27:41 UTC) #3
hans
On 2015/12/31 16:27:41, Nico wrote: > hans for all the clang tot bots in chromium.fyi.json ...
4 years, 11 months ago (2016-01-04 11:10:49 UTC) #4
Nico
martiniss, dpranke: ping
4 years, 11 months ago (2016-01-14 19:01:08 UTC) #5
Nico
martiniss, dpranke: ping^2
4 years, 11 months ago (2016-01-19 17:46:53 UTC) #6
Nico
martiniss, dpranke: ping^2
4 years, 11 months ago (2016-01-19 17:46:55 UTC) #7
Dirk Pranke
The code changes look mechanically fine, but I don't know if we have the capacity ...
4 years, 11 months ago (2016-01-19 17:51:44 UTC) #9
martiniss
On 2016/01/19 at 17:51:44, dpranke wrote: > The code changes look mechanically fine, but I ...
4 years, 11 months ago (2016-01-19 20:05:57 UTC) #10
ghost stip (do not use)
https://chromiumcodereview.appspot.com/1551183002/diff/1/testing/buildbot/chromium.android.json File testing/buildbot/chromium.android.json (right): https://chromiumcodereview.appspot.com/1551183002/diff/1/testing/buildbot/chromium.android.json#newcode160 testing/buildbot/chromium.android.json:160: "can_use_on_swarming_builders": true this will "work" in that it will ...
4 years, 11 months ago (2016-01-19 20:20:33 UTC) #11
martiniss
Also, you probably don't want to swarm the Android Cloud builder. Talk to bpastene https://codereview.chromium.org/1551183002/diff/1/testing/buildbot/chromium.fyi.json ...
4 years, 11 months ago (2016-01-20 01:07:01 UTC) #13
Nico
https://codereview.chromium.org/1551183002/diff/1/testing/buildbot/chromium.fyi.json File testing/buildbot/chromium.fyi.json (right): https://codereview.chromium.org/1551183002/diff/1/testing/buildbot/chromium.fyi.json#newcode2 testing/buildbot/chromium.fyi.json:2: "Android Cloud Tests": { On 2016/01/20 01:07:01, martiniss wrote: ...
4 years, 11 months ago (2016-01-20 02:13:18 UTC) #14
Dirk Pranke
On 2016/01/20 02:13:18, Nico wrote: > If things shouldn't be swarmed on some bot, you ...
4 years, 11 months ago (2016-01-20 18:53:23 UTC) #15
Nico
(Kind of forgot about this; will break it up.) On 2016/01/20 18:53:23, Dirk Pranke wrote: ...
4 years, 10 months ago (2016-01-28 20:20:55 UTC) #16
Nico
https://codereview.chromium.org/1551183002/diff/1/testing/buildbot/chromium.memory.json File testing/buildbot/chromium.memory.json (right): https://codereview.chromium.org/1551183002/diff/1/testing/buildbot/chromium.memory.json#newcode531 testing/buildbot/chromium.memory.json:531: "can_use_on_swarming_builders": true On 2016/01/19 20:20:33, stip wrote: > for ...
4 years, 10 months ago (2016-01-28 20:21:33 UTC) #17
Dirk Pranke
On 2016/01/28 20:20:55, Nico wrote: > (Kind of forgot about this; will break it up.) ...
4 years, 10 months ago (2016-01-28 23:39:21 UTC) #18
Dirk Pranke
closing ... I think we ended up migrating everything in other CLs?
4 years, 3 months ago (2016-08-31 23:12:22 UTC) #19
Nico
4 years, 3 months ago (2016-08-31 23:14:03 UTC) #20
Message was sent while issue was closed.
Yes, maruel++ did this a few weeks ago.

Powered by Google App Engine
This is Rietveld 408576698