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

Issue 2692133002: chromium.perf.json: Special case some desktop bots (Closed)

Created:
3 years, 10 months ago by martiniss
Modified:
3 years, 10 months ago
Reviewers:
nednguyen, dtu, sullivan
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

chromium.perf.json: Special case some desktop bots This should help alleviate capacity issues we've been having for a bit. Longer-term solutions should be coming for this, but these particular bots are a bit overloaded right now. PERF SHERIFFS: This CL will cause changes to the following benchmarks * page_cycler_v2.intl_hi_ru on Mac 10.12 Perf * blink_perf.parser on Mac 10.12 Perf It also affects two other benchmarks, but those are already not sending data, because of missing capacity, so those won't cause any jumps in timings. BUG=687425, 680360, 685275 Review-Url: https://codereview.chromium.org/2692133002 Cr-Commit-Position: refs/heads/master@{#450159} Committed: https://chromium.googlesource.com/chromium/src/+/549d1501a66ed2ec0283149f65c71d64569183aa

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M testing/buildbot/chromium.perf.json View 8 chunks +8 lines, -8 lines 0 comments Download

Messages

Total messages: 9 (5 generated)
martiniss
PTAL
3 years, 10 months ago (2017-02-13 22:02:02 UTC) #2
sullivan
lgtm Would be great to add the other blocking bugs for crbug.com/691582 that this should ...
3 years, 10 months ago (2017-02-13 22:04:05 UTC) #3
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/2692133002/1
3 years, 10 months ago (2017-02-13 22:08:50 UTC) #6
commit-bot: I haz the power
3 years, 10 months ago (2017-02-13 23:57:30 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/549d1501a66ed2ec0283149f65c7...

Powered by Google App Engine
This is Rietveld 408576698