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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 594153003: Revert of Enable isolate on Linux release builder. (Closed)

Created:
6 years, 3 months ago by jam
Modified:
6 years, 3 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews, pgervais+watch_chromium.org, kjellander-cc_chromium.org, cmp-cc_chromium.org, stip+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Project:
tools
Visibility:
Public.

Description

Revert of Enable isolate on Linux release builder. (patchset #3 id:40001 of https://codereview.chromium.org/595143002/) Reason for revert: breaks swarming on bots Original issue's description: > Enable isolate on Linux release builder. > > Fix coverage warning by removing old codepath. The two parameter version of find_isolated_tests has been called by GPU bots for a long time so we know it works. > > BUG=414808 > > Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292114 TBR=maruel@chromium.org NOTREECHECKS=true NOTRY=true BUG=414808 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292115

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -144 lines) Patch
M scripts/slave/recipe_modules/chromium/chromium_linux.py View 2 chunks +0 lines, -3 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_tests/api.py View 1 chunk +4 lines, -1 line 0 comments Download
M scripts/slave/recipes/chromium.expected/archive_dependencies_failure.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/arm.json View 1 chunk +20 lines, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/buildnumber_zero.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/dynamic_gtest.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/dynamic_gtest_on_builder.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_fyi_Linux_ARM_Cross_Compile.json View 1 chunk +20 lines, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Linux_Builder.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Linux_Builder__dbg_.json View 1 chunk +20 lines, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Linux_Tests.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_client_skia_Linux_Builder.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_client_skia_Linux_Tests.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_tryserver_chromium_linux_bisect_builder.json View 1 chunk +1 line, -14 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/one_failure_keeps_going_dynamic_tests.json View 1 chunk +1 line, -14 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jam
Created Revert of Enable isolate on Linux release builder.
6 years, 3 months ago (2014-09-24 16:38:29 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/594153003/1
6 years, 3 months ago (2014-09-24 16:38:37 UTC) #2
commit-bot: I haz the power
6 years, 3 months ago (2014-09-24 16:39:21 UTC) #3
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as 292115

Powered by Google App Engine
This is Rietveld 408576698