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

Issue 2708373003: Regenerate perf JSON now that battor.power_cases_no_chrome_trace is gone (Closed)

Created:
3 years, 10 months ago by charliea (OOO until 10-5)
Modified:
3 years, 10 months ago
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Regenerate perf JSON now that battor.power_cases_no_chrome_trace is gone I didn't realize I had to do this earlier, and now it's causing problems on bots. BUG=695156 Review-Url: https://codereview.chromium.org/2708373003 Cr-Commit-Position: refs/heads/master@{#452539} Committed: https://chromium.googlesource.com/chromium/src/+/ee09c859d3f509fc33d7c9e3b3ffe0959644f9e2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4340 lines, -1091 lines) Patch
M testing/buildbot/chromium.perf.json View 40 chunks +3656 lines, -920 lines 0 comments Download
M testing/buildbot/chromium.perf.fyi.json View 6 chunks +684 lines, -171 lines 0 comments Download

Messages

Total messages: 25 (11 generated)
charliea (OOO until 10-5)
3 years, 10 months ago (2017-02-22 20:12:10 UTC) #2
nednguyen
On 2017/02/22 20:12:10, charliea wrote: lgtm
3 years, 10 months ago (2017-02-22 20:12:36 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/2708373003/1
3 years, 10 months ago (2017-02-22 20:16:25 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/370457)
3 years, 10 months ago (2017-02-22 20:26:40 UTC) #7
nednguyen
On 2017/02/22 20:26:40, commit-bot: I haz the power wrote: > Try jobs failed on following ...
3 years, 10 months ago (2017-02-22 20:39:17 UTC) #8
charliea (OOO until 10-5)
dpranke@, could you put a quick stamp on this?
3 years, 10 months ago (2017-02-23 14:59:34 UTC) #10
charliea (OOO until 10-5)
Removing dpranke@ (just saw that sullivan@ is also an owner)
3 years, 10 months ago (2017-02-23 15:01:24 UTC) #12
sullivan
lgtm
3 years, 10 months ago (2017-02-23 15:02:04 UTC) #13
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/2708373003/1
3 years, 10 months ago (2017-02-23 15:02:49 UTC) #15
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/371250)
3 years, 10 months ago (2017-02-23 15:08:38 UTC) #17
nednguyen
On 2017/02/23 15:08:38, commit-bot: I haz the power wrote: > Try jobs failed on following ...
3 years, 10 months ago (2017-02-23 15:09:17 UTC) #18
Dirk Pranke
lgtm
3 years, 10 months ago (2017-02-23 17:07:26 UTC) #20
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/2708373003/1
3 years, 10 months ago (2017-02-23 17:35:06 UTC) #22
commit-bot: I haz the power
3 years, 10 months ago (2017-02-23 17:39:36 UTC) #25
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ee09c859d3f509fc33d7c9e3b3ff...

Powered by Google App Engine
This is Rietveld 408576698