aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBob Haarman <inglorion@chromium.org>2022-09-29 17:23:51 -0700
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2022-09-30 20:48:48 +0000
commit3d3b092864634dd7e7d342231cdc8e23c829888d (patch)
treefc032232c770356ab694657ee7912238e96558a5
parent17d9a5c3229a2033a834eaf6284694683bb400a0 (diff)
downloadtoolchain-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.py10
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":