diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2024-02-05 15:40:19 +0000 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2024-02-07 14:58:53 +0100 |
| commit | c645bac4e06bf9642cc8e339d027a5d6ec54d811 (patch) | |
| tree | 467a5767ec71f0591045791b9ddf8ce84c62d82f /tests | |
| parent | 1604c0493193273e4eac547f86fbd2845e7f9af4 (diff) | |
| download | focaccia-qemu-c645bac4e06bf9642cc8e339d027a5d6ec54d811.tar.gz focaccia-qemu-c645bac4e06bf9642cc8e339d027a5d6ec54d811.zip | |
iotests: fix leak of tmpdir in dry-run mode
Creating an instance of the 'TestEnv' class will create a temporary directory. This dir is only deleted, however, in the __exit__ handler invoked by a context manager. In dry-run mode, we don't use the TestEnv via a context manager, so were leaking the temporary directory. Since meson invokes 'check' 5 times on each configure run, developers /tmp was filling up with empty temporary directories. Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> Message-ID: <20240205154019.1841037-1-berrange@redhat.com> Reviewed-by: Michael Tokarev <mjt@tls.msk.ru> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'tests')
| -rwxr-xr-x | tests/qemu-iotests/check | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/qemu-iotests/check b/tests/qemu-iotests/check index f2e9d27dcf..56d88ca423 100755 --- a/tests/qemu-iotests/check +++ b/tests/qemu-iotests/check @@ -184,7 +184,8 @@ if __name__ == '__main__': sys.exit(str(e)) if args.dry_run: - print('\n'.join([os.path.basename(t) for t in tests])) + with env: + print('\n'.join([os.path.basename(t) for t in tests])) else: with TestRunner(env, tap=args.tap, color=args.color) as tr: |