diff options
author | Zhizhou Yang <zhizhouy@google.com> | 2020-01-22 15:55:19 -0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-01-24 02:20:35 +0000 |
commit | 0ee6a5701bedfa6b67b33f8b47cd705d713b1777 (patch) | |
tree | c17f371ac0af1242e3a2737f0cffbc86e3f3e8bf /crosperf | |
parent | c2dec89d0d9758bcbb2457a04f4bf49d5bf4731c (diff) | |
download | toolchain-utils-0ee6a5701bedfa6b67b33f8b47cd705d713b1777.tar.gz |
crosperf: remove AFE server interactions
This patch removes afe locking mode for crosperf, since:
1) All lab machines has been migrated to skylab.
2) Interaction with afe depends on python 2 code in autotest.
BUG=chromium:1044764
TEST=Passed unittest and simple experiment run.
Change-Id: Ie0a250548d739f32bbc08619f637380104df6c28
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2016008
Commit-Queue: Zhizhou Yang <zhizhouy@google.com>
Tested-by: Zhizhou Yang <zhizhouy@google.com>
Auto-Submit: Zhizhou Yang <zhizhouy@google.com>
Reviewed-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'crosperf')
-rw-r--r-- | crosperf/experiment_runner.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/crosperf/experiment_runner.py b/crosperf/experiment_runner.py index a536f161..39e3f863 100644 --- a/crosperf/experiment_runner.py +++ b/crosperf/experiment_runner.py @@ -103,7 +103,7 @@ class ExperimentRunner(object): """Get where is the machine from. Returns: - The location of the machine: local, skylab or afe + The location of the machine: local or skylab """ # We assume that lab machine always starts with chromeos*, and local # machines are ip address. @@ -111,7 +111,7 @@ class ExperimentRunner(object): if lock_mgr.CheckMachineInSkylab(machine): return 'skylab' else: - return 'afe' + raise RuntimeError('Lab machine not in Skylab.') return 'local' def _LockAllMachines(self, experiment): @@ -122,7 +122,6 @@ class ExperimentRunner(object): from being able to update/use the machines while this experiment is running: - Skylab machines: Use skylab lease-dut mechanism to lease - - AFE machines: Use AFE lock mechanism to lock - Local machines: Use file lock mechanism to lock """ if test_flag.GetTestMode(): |