Re: [PATCH] scripts: display how long each test takes to execute

[Thomas Huth]

On 14/09/2020 13.09, Daniel P. Berrangé wrote:
> Sometimes under CI tests non-deterministically take longer to execute
> than expected which can trigger timeouts. It is almost impossible to
> diagnose this though without seeing execution time for each test case.
> 
> With this change, when passing "V=1" to make, we get a duration printed
> at test completion:
> 
> $ make check V=1
> ...snip...
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))} 
> QTEST_QEMU_IMG=./qemu-img 
> G_TEST_DBUS_DAEMON=/home/berrange/src/virt/qemu/tests/dbus-vmstate-daemon.sh 
> QTEST_QEMU_BINARY=./qemu-system-x86_64 tests/qtest/migration-test --tap -k
> PASS 1 qtest-x86_64: migration-test /x86_64/migration/deprecated
> PASS 2 qtest-x86_64: migration-test /x86_64/migration/bad_dest
> PASS 3 qtest-x86_64: migration-test /x86_64/migration/fd_proto
> PASS 4 qtest-x86_64: migration-test /x86_64/migration/validate_uuid
> PASS 5 qtest-x86_64: migration-test /x86_64/migration/validate_uuid_error
> PASS 6 qtest-x86_64: migration-test 
> /x86_64/migration/validate_uuid_src_not_set
> PASS 7 qtest-x86_64: migration-test 
> /x86_64/migration/validate_uuid_dst_not_set
> PASS 8 qtest-x86_64: migration-test /x86_64/migration/auto_converge
> PASS 9 qtest-x86_64: migration-test /x86_64/migration/postcopy/unix
> PASS 10 qtest-x86_64: migration-test /x86_64/migration/postcopy/recovery
> PASS 11 qtest-x86_64: migration-test /x86_64/migration/precopy/unix
> PASS 12 qtest-x86_64: migration-test /x86_64/migration/precopy/tcp
> PASS 13 qtest-x86_64: migration-test /x86_64/migration/xbzrle/unix
> PASS 14 qtest-x86_64: migration-test /x86_64/migration/multifd/tcp/none
> PASS 15 qtest-x86_64: migration-test /x86_64/migration/multifd/tcp/cancel
> PASS 16 qtest-x86_64: migration-test /x86_64/migration/multifd/tcp/zlib
> TIME 28 seconds

If I run "make check -j8" in parallel, it's quite hard to match the "TIME xx
seconds" lines with the corresponding tests... so could you please in
include the test name in the output there?

 Thanks,
  Thomas