[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 5/8] iotests: connect stdin to /dev/null when running tests
From: |
Daniel P . Berrangé |
Subject: |
[PATCH v2 5/8] iotests: connect stdin to /dev/null when running tests |
Date: |
Fri, 3 Mar 2023 16:07:24 +0000 |
Currently the tests have their stdin inherited from the test harness,
meaning they are connected to a TTY. The QEMU processes spawned by
certain tests, however, modify TTY settings and if the test exits
abnormally the settings might not be restored.
The python test harness thus has some logic which will capture the
initial TTY settings and restore them once all tests are finished.
This does not, however, take into account the possibility of many
copies of the 'check' program running in parallel. With parallel
execution, a later invokation may save the TTY state that QEMU has
already modified, and thus restore bad state leaving the TTY
non-functional.
None of the I/O tests shnould actually be interactive requiring
user input and so they should not require a TTY at all. To avoid
this while TTY save/restore complexity we can connect the test
stdin to /dev/null instead.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
tests/qemu-iotests/testrunner.py | 22 ++--------------------
1 file changed, 2 insertions(+), 20 deletions(-)
diff --git a/tests/qemu-iotests/testrunner.py b/tests/qemu-iotests/testrunner.py
index e734800b3d..81519ed6e2 100644
--- a/tests/qemu-iotests/testrunner.py
+++ b/tests/qemu-iotests/testrunner.py
@@ -24,12 +24,10 @@
import subprocess
import contextlib
import json
-import termios
import shutil
import sys
from multiprocessing import Pool
-from contextlib import contextmanager
-from typing import List, Optional, Iterator, Any, Sequence, Dict, \
+from typing import List, Optional, Any, Sequence, Dict, \
ContextManager
from testenv import TestEnv
@@ -56,22 +54,6 @@ def file_diff(file1: str, file2: str) -> List[str]:
return res
-# We want to save current tty settings during test run,
-# since an aborting qemu call may leave things screwed up.
-@contextmanager
-def savetty() -> Iterator[None]:
- isterm = sys.stdin.isatty()
- if isterm:
- fd = sys.stdin.fileno()
- attr = termios.tcgetattr(fd)
-
- try:
- yield
- finally:
- if isterm:
- termios.tcsetattr(fd, termios.TCSADRAIN, attr)
-
-
class LastElapsedTime(ContextManager['LastElapsedTime']):
""" Cache for elapsed time for tests, to show it during new test run
@@ -169,7 +151,6 @@ def __enter__(self) -> 'TestRunner':
self._stack = contextlib.ExitStack()
self._stack.enter_context(self.env)
self._stack.enter_context(self.last_elapsed)
- self._stack.enter_context(savetty())
return self
def __exit__(self, exc_type: Any, exc_value: Any, traceback: Any) -> None:
@@ -294,6 +275,7 @@ def do_run_test(self, test: str, mp: bool) -> TestResult:
t0 = time.time()
with f_bad.open('w', encoding="utf-8") as f:
with subprocess.Popen(args, cwd=str(f_test.parent), env=env,
+ stdin=subprocess.DEVNULL,
stdout=f, stderr=subprocess.STDOUT) as proc:
try:
proc.wait()
--
2.39.2
- [PATCH v2 0/8] iotests: make meson aware of individual I/O tests, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 1/8] iotests: explicitly pass source/build dir to 'check' command, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 2/8] iotests: allow test discovery before building, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 3/8] iotests: strip subdir path when listing tests, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 4/8] iotests: print TAP protocol version when reporting tests, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 5/8] iotests: connect stdin to /dev/null when running tests,
Daniel P . Berrangé <=
- [PATCH v2 7/8] iotests: register each I/O test separately with meson, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 6/8] iotests: always use a unique sub-directory per test, Daniel P . Berrangé, 2023/03/03
- [PATCH v2 8/8] iotests: remove the check-block.sh script, Daniel P . Berrangé, 2023/03/03