diff options
author | simonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-24 22:17:30 +0000 |
---|---|---|
committer | simonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-24 22:17:30 +0000 |
commit | e6233edfaa21b5fc7211e46d59c437e5781271c7 (patch) | |
tree | 86c2980e1c5e029ac26dad968cae453e0ff3e8fa /tools/run-bisect-perf-regression.py | |
parent | f37579e600aab07b8e5d5f0489b47b6aa7cff68a (diff) | |
download | chromium_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-x | tools/run-bisect-perf-regression.py | 6 |
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 = '' |