]> git.proxmox.com Git - mirror_qemu.git/commit
iotests/testrunner: Flush after run_test()
authorHanna Reitz <hreitz@redhat.com>
Fri, 6 May 2022 13:42:15 +0000 (15:42 +0200)
committerKevin Wolf <kwolf@redhat.com>
Thu, 12 May 2022 10:25:18 +0000 (12:25 +0200)
commit22d92e71c77c8690995315c9ef5a1136c6300047
treef6b9954722993b70685997382fe60418327f45cf
parent9ec7a59b5aad4b736871c378d30f5ef5ec51cb52
iotests/testrunner: Flush after run_test()

When stdout is not a terminal, the buffer may not be flushed at each end
of line, so we should flush after each test is done.  This is especially
apparent when run by check-block, in two ways:

First, when running make check-block -jX with X > 1, progress indication
was missing, even though testrunner.py does theoretically print each
test's status once it has been run, even in multi-processing mode.
Flushing after each test restores this progress indication.

Second, sometimes make check-block failed altogether, with an error
message that "too few tests [were] run".  I presume that's because one
worker process in the job pool did not get to flush its stdout before
the main process exited, and so meson did not get to see that worker's
test results.  In any case, by flushing at the end of run_test(), the
problem has disappeared for me.

Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Message-Id: <20220506134215.10086-1-hreitz@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
tests/qemu-iotests/testrunner.py