[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] scripts: display how long each test takes to execute
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH] scripts: display how long each test takes to execute |
Date: |
Mon, 12 Oct 2020 16:09:16 +0100 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Mon, Oct 12, 2020 at 05:07:18PM +0200, Thomas Huth wrote:
> 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?
I wasn't intending to update this given paolo's negative response to
the patch.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|