diff options
| author | Caroline Tice <cmtice@google.com> | 2016-03-03 10:43:56 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-03-05 00:08:47 +0000 |
| commit | 25d32b4add9a0d0ac4ec682062e45baea97bff04 (patch) | |
| tree | 80fe6497685a0e53417f2a58834de1cd08cc22de /crosperf/benchmark_run.py | |
| parent | eff2fc1c86365c53ae228df4653524ceb6168d92 (diff) | |
| download | toolchain-utils-25d32b4add9a0d0ac4ec682062e45baea97bff04.tar.gz | |
[crosperf] Fix missing chrome_version problem (v2).
When generating the json reports, occasionally the chrome_version
comes up blank (this can happen when reading some results from the
cache, for example). A blank chrome_version messes up our dremel
database scripts. This CL updates the read-cache scenario to find
the chrome_version in the cache keys (human-readable) file.
BUG=chromium:591494
TEST=Tested with experiment that was generating blank Chrome
versions and this fixed the problem.
Change-Id: I29985581aa00bc30fb65720fb9d197f1475445e4
Reviewed-on: https://chrome-internal-review.googlesource.com/249960
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Diffstat (limited to 'crosperf/benchmark_run.py')
| -rw-r--r-- | crosperf/benchmark_run.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/crosperf/benchmark_run.py b/crosperf/benchmark_run.py index 209b3f71..35977a63 100644 --- a/crosperf/benchmark_run.py +++ b/crosperf/benchmark_run.py @@ -112,9 +112,12 @@ class BenchmarkRun(threading.Thread): self.machine) self.cache.StoreResult(self.result) - if self.machine and not self.label.chrome_version: - self.label.chrome_version = self.machine_manager.GetChromeVersion( - self.machine) + if not self.label.chrome_version: + if self.machine: + self.label.chrome_version = self.machine_manager.GetChromeVersion( + self.machine) + elif self.result.chrome_version: + self.label.chrome_version = self.result.chrome_version if self.terminated: return |