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

Issue 1907463002: Disable rasterize_and_record_micro.top_25_smooth on Linux and Android (Closed)

Created:
4 years, 8 months ago by petrcermak
Modified:
4 years, 8 months ago
Reviewers:
Sami, Sami (do not use)
CC:
chromium-reviews, telemetry-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Disable rasterize_and_record_micro.top_25_smooth on Linux and Android Rationale: The benchmark is causing persistent redness on one Linux and several Android perf bots: https://build.chromium.org/p/chromium.perf/builders/Linux%20Perf%20(5) https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus5%20Perf%20(2) https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus5X%20Perf%20(2) https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus7v2%20Perf%20(2) https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus6%20Perf%20(2) https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus9%20Perf%20(2) https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus9%20Perf%20(2) TBR=skyostil BUG=600377, 605120 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq Committed: https://crrev.com/502afb658d9b946b123ced6d0c91c444adb23f19 Cr-Commit-Position: refs/heads/master@{#388528}

Patch Set 1 #

Patch Set 2 : Disable on Android as well #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M tools/perf/benchmarks/rasterize_and_record_micro.py View 1 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 21 (12 generated)
petrcermak
PTAL. Thanks, Petr
4 years, 8 months ago (2016-04-20 11:14:44 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1907463002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1907463002/1
4 years, 8 months ago (2016-04-20 11:14:57 UTC) #4
Sami
lgtm.
4 years, 8 months ago (2016-04-20 11:23:24 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: android_s5_perf_cq on tryserver.chromium.perf (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_cq/builds/154)
4 years, 8 months ago (2016-04-20 14:15:49 UTC) #8
petrcermak
Looks like I'll have to disable the benchmark on both Linux and Android :-\
4 years, 8 months ago (2016-04-20 14:17:29 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1907463002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1907463002/20001
4 years, 8 months ago (2016-04-20 15:15:18 UTC) #16
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 8 months ago (2016-04-20 17:43:50 UTC) #18
Sami
A revert of this CL (patchset #2 id:20001) has been created in https://codereview.chromium.org/1903903003/ by skyostil@chromium.org. ...
4 years, 8 months ago (2016-04-21 10:10:26 UTC) #19
commit-bot: I haz the power
4 years, 8 months ago (2016-04-22 19:24:41 UTC) #21
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/502afb658d9b946b123ced6d0c91c444adb23f19
Cr-Commit-Position: refs/heads/master@{#388528}

Powered by Google App Engine
This is Rietveld 408576698