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

Issue 2291703003: Revert of V8: increase number of shards for arm64 gc stress. (Closed)

Created:
4 years, 3 months ago by Michael Achenbach
Modified:
4 years, 3 months ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org, Michael Achenbach, Benedikt Meurer
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Revert of V8: increase number of shards for arm64 gc stress. (patchset #1 id:1 of https://codereview.chromium.org/2248363003/ ) Reason for revert: The root cause of the slow tests has been fixed. A shard increase is not needed. Original issue's description: > V8: increase number of shards for arm64 gc stress. > > Reason: shard#0 is timing out on swarming. > Expected follow up: if this helps, shard in other > builders as well. > > R=hablich@chromium.org > CC=machenbach@chromium.org > BUG=638295 > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/a415f195afcd9db6c89a5da50df939e76d69b849 TBR=hablich@chromium.org,tandrii@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=638295 Committed: https://chromium.googlesource.com/chromium/tools/build/+/860d8d2ab8b6ad3a5ca09d23356252b4944624cc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -14 lines) Patch
M scripts/slave/recipe_modules/v8/builders.py View 2 chunks +1 line, -2 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm64___sim___gc_stress.json View 3 chunks +6 lines, -12 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Michael Achenbach
Created Revert of V8: increase number of shards for arm64 gc stress.
4 years, 3 months ago (2016-08-30 06:57:26 UTC) #2
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/2291703003/1
4 years, 3 months ago (2016-08-30 06:57:38 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/tools/build/+/860d8d2ab8b6ad3a5ca09d23356252b4944624cc
4 years, 3 months ago (2016-08-30 07:01:13 UTC) #5
tandrii(chromium)
4 years, 3 months ago (2016-08-30 09:18:05 UTC) #6
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698