summaryrefslogtreecommitdiffstats
path: root/tools/run-bisect-perf-regression.py
diff options
context:
space:
mode:
authorsimonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-10-24 22:17:30 +0000
committersimonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-10-24 22:17:30 +0000
commite6233edfaa21b5fc7211e46d59c437e5781271c7 (patch)
tree86c2980e1c5e029ac26dad968cae453e0ff3e8fa /tools/run-bisect-perf-regression.py
parentf37579e600aab07b8e5d5f0489b47b6aa7cff68a (diff)
downloadchromium_src-e6233edfaa21b5fc7211e46d59c437e5781271c7.zip
chromium_src-e6233edfaa21b5fc7211e46d59c437e5781271c7.tar.gz
chromium_src-e6233edfaa21b5fc7211e46d59c437e5781271c7.tar.bz2
Fixed link step in output for perf bot. Previously, wasn't parsing out the url of the cloud file properly, instead was taking the whole line.
Broken: http://build.chromium.org/p/tryserver.chromium/builders/linux_perf_bisect/builds/441 Fixed: http://build.chromium.org/p/tryserver.chromium/builders/linux_perf_bisect/builds/449 BUG= NOTRY=true Review URL: https://codereview.chromium.org/41573002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230842 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/run-bisect-perf-regression.py')
-rwxr-xr-xtools/run-bisect-perf-regression.py6
1 files changed, 6 insertions, 0 deletions
diff --git a/tools/run-bisect-perf-regression.py b/tools/run-bisect-perf-regression.py
index e165fed..8af0fa3 100755
--- a/tools/run-bisect-perf-regression.py
+++ b/tools/run-bisect-perf-regression.py
@@ -211,6 +211,12 @@ def _RunPerformanceTest(config, path_to_file):
cloud_file_link = [t for t in output.splitlines()
if 'storage.googleapis.com/chromium-telemetry/html-results/' in t]
if cloud_file_link:
+ # What we're getting here is basically "View online at http://..." so parse
+ # out just the url portion.
+ cloud_file_link = cloud_file_link[0]
+ cloud_file_link = [t for t in cloud_file_link.split(' ')
+ if 'storage.googleapis.com/chromium-telemetry/html-results/' in t]
+ assert cloud_file_link, "Couldn't parse url from output."
cloud_file_link = cloud_file_link[0]
else:
cloud_file_link = ''