| Message ID | 20210512214642.2803189-10-jsnow@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Python: delint iotests, machine.py and console_socket.py | expand |
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py index 5d5ec40429b..e09c991b84e 100644 --- a/tests/qemu-iotests/iotests.py +++ b/tests/qemu-iotests/iotests.py @@ -311,7 +311,7 @@ def qemu_nbd_popen(*args): cmd.extend(args) log('Start NBD server') - p = subprocess.Popen(cmd) + p = subprocess.Popen(cmd) # pylint: disable=consider-using-with try: while not os.path.exists(pid_file): if p.poll() is not None: diff --git a/tests/qemu-iotests/testrunner.py b/tests/qemu-iotests/testrunner.py index 1fc61fcaa34..34fb551c01b 100644 --- a/tests/qemu-iotests/testrunner.py +++ b/tests/qemu-iotests/testrunner.py @@ -258,6 +258,7 @@ def do_run_test(self, test: str) -> TestResult: t0 = time.time() with f_bad.open('w', encoding="utf-8") as f: + # pylint: disable=consider-using-with proc = subprocess.Popen(args, cwd=str(f_test.parent), env=env, stdout=f, stderr=subprocess.STDOUT) try:
In a few cases, we can't use 'with ...' because they belong to long-running classes that need those items to stay open at the end of the block. We're handling it, so tell pylint to shush. Signed-off-by: John Snow <jsnow@redhat.com> --- tests/qemu-iotests/iotests.py | 2 +- tests/qemu-iotests/testrunner.py | 1 + 2 files changed, 2 insertions(+), 1 deletion(-)