diff options
author | Zhizhou Yang <zhizhouy@google.com> | 2019-12-04 17:17:18 -0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-12-12 13:36:11 +0000 |
commit | f4fe66ea31f71b54b18a4e41acb936b500837f30 (patch) | |
tree | 57320fa77e570027a4370a4c6de930d14feee78b /crosperf | |
parent | 5a74f07b374010951c7c66226cbbb548de69789e (diff) | |
download | toolchain-utils-f4fe66ea31f71b54b18a4e41acb936b500837f30.tar.gz |
Crosperf: Do not image DUT in Skylab mode
We should not image DUT with certain image when the test is going to be
launched via 'skylab create-test'. This is because the DUT may be
occupied by other tasks before Crosperf acutally get it assigned, and
the first step after the test starts is to flash image.
This patch is a temproray solution to ensure no bad behavior happens on
the DUT before testing.
TEST=Tested wth skylab run, w/o caching.
BUG=chromium:984790
Change-Id: Ie2f439d5777813905bdca5f4a2d1e401df2dc83e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1952172
Tested-by: Zhizhou Yang <zhizhouy@google.com>
Reviewed-by: Caroline Tice <cmtice@chromium.org>
Commit-Queue: Zhizhou Yang <zhizhouy@google.com>
Auto-Submit: Zhizhou Yang <zhizhouy@google.com>
Diffstat (limited to 'crosperf')
-rw-r--r-- | crosperf/schedv2.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/crosperf/schedv2.py b/crosperf/schedv2.py index 9a4f9660..68e1e5b8 100644 --- a/crosperf/schedv2.py +++ b/crosperf/schedv2.py @@ -108,6 +108,11 @@ class DutWorker(Thread): if self._terminated: return 1 + if self._sched.get_experiment().skylab: + self._logger.LogOutput('Skylab mode, do not image before testing.') + self._dut.label = label + return 0 + self._logger.LogOutput('Reimaging {} using {}'.format(self, label)) self._stat_num_reimage += 1 self._stat_annotation = 'reimaging using "{}"'.format(label.name) |