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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2295803002: Fixing syntax for --upload-results flag (Closed)

Created:
4 years, 3 months ago by eyaich1
Modified:
4 years, 3 months ago
CC:
chromium-reviews, telemetry-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Fixing syntax for --upload-results flag BUG=chromium:633253 Committed: https://crrev.com/f442527a1df5086ff5cdf99faba1fb02bb69ee10 Cr-Commit-Position: refs/heads/master@{#415520}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+925 lines, -753 lines) Patch
M testing/buildbot/chromium.perf.fyi.json View 752 chunks +924 lines, -752 lines 0 comments Download
M tools/perf/generate_perf_json.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 15 (6 generated)
eyaich1
4 years, 3 months ago (2016-08-30 14:25:50 UTC) #2
dtu
lgtm
4 years, 3 months ago (2016-08-30 18:14:28 UTC) #4
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/2295803002/1
4 years, 3 months ago (2016-08-30 18:15:02 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/249414)
4 years, 3 months ago (2016-08-30 18:24:11 UTC) #7
eyaich1
Ken can you please give a quick pass since we need approval from someone in ...
4 years, 3 months ago (2016-08-30 18:35:26 UTC) #9
Ken Russell (switch to Gerrit)
lgtm
4 years, 3 months ago (2016-08-30 19:36:37 UTC) #10
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/2295803002/1
4 years, 3 months ago (2016-08-30 23:45:39 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-08-31 00:39:13 UTC) #13
commit-bot: I haz the power
4 years, 3 months ago (2016-08-31 00:40:53 UTC) #15
Message was sent while issue was closed. 
Patchset 1 (id:??) landed as
https://crrev.com/f442527a1df5086ff5cdf99faba1fb02bb69ee10
Cr-Commit-Position: refs/heads/master@{#415520}

Powered by Google App Engine
This is Rietveld 408576698