diff options
author | Bob Haarman <inglorion@chromium.org> | 2022-09-29 17:23:51 -0700 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-09-30 20:48:48 +0000 |
commit | 3d3b092864634dd7e7d342231cdc8e23c829888d (patch) | |
tree | fc032232c770356ab694657ee7912238e96558a5 | |
parent | 17d9a5c3229a2033a834eaf6284694683bb400a0 (diff) | |
download | toolchain-utils-3d3b092864634dd7e7d342231cdc8e23c829888d.tar.gz |
crosperf: fix pylint errors in suite_runner.py
crosperf/suit_runner.py uses .format(), which we no longer want.
This converts the code to use f-strings instead.
BUG=None
TEST=See that pylint no longer complains about .format()
Change-Id: I7085f2da5659843d2501eb2f19137b64dff7b106
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3928353
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Reviewed-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
-rw-r--r-- | crosperf/suite_runner.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/crosperf/suite_runner.py b/crosperf/suite_runner.py index 9fc99138..dfd14b21 100644 --- a/crosperf/suite_runner.py +++ b/crosperf/suite_runner.py @@ -50,7 +50,7 @@ def GetProfilerArgs(profiler_args): def GetDutConfigArgs(dut_config): - return "dut_config={}".format(pipes.quote(json.dumps(dut_config))) + return f"dut_config={pipes.quote(json.dumps(dut_config))}" class SuiteRunner(object): @@ -222,10 +222,10 @@ class SuiteRunner(object): # process namespace and we can kill process created easily by their # process group. chrome_root_options = ( - "--no-ns-pid " - "--chrome_root={0} --chrome_root_mount={1} " - 'FEATURES="-usersandbox" ' - "CHROME_ROOT={1}".format(label.chrome_src, CHROME_MOUNT_DIR) + f"--no-ns-pid " + f"--chrome_root={label.chrome_src} --chrome_root_mount={CHROME_MOUNT_DIR} " + f'FEATURES="-usersandbox" ' + f"CHROME_ROOT={CHROME_MOUNT_DIR}" ) if self.log_level != "verbose": |