[PATCH 2/2] tests: commandtest: Make 'test4' checking daemonization more reliable

Ján Tomko jtomko at redhat.com
Thu Jul 23 08:16:22 UTC 2020


On a Tuesday in 2020, Peter Krempa wrote:
>The 'commandhelper' checks effectively whether the parent process is
>still around to report whether it was daemonized or not.
>
>This creates a unlikely race condition in cases when we do actually
>daemonize the process as the intermediate process used for the
>daemonization might not have terminated yet which would report wrong
>result leading to test failure.
>
>For now there's just 'test4' which actually daemonizes the process.
>
>Add an argument '--check-daemonize' which asks for retires of the

:%s/retires/retries/g

>daemonization check in cases where we expect that the commandhelper is
>going to be daemonized and use it in 'test4' to make the test more
>reliable.
>
>I've observed the test failure sporradically when my box is more loaded

:%s/sporradically/sporadically/g

( justone r, like the islands https://en.wikipedia.org/wiki/Sporades )


Also:

:%s/more loaded/under load/g

or

:%s/more loaded/under heavy load/g

>e.g. while building two trees at once.
>
>Signed-off-by: Peter Krempa <pkrempa at redhat.com>
>---
> tests/commanddata/test4.log |  1 +
> tests/commandhelper.c       | 17 ++++++++++++++++-
> tests/commandtest.c         |  3 ++-
> 3 files changed, 19 insertions(+), 2 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko at redhat.com>

Thank you for fixing this.

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200723/d4856dfd/attachment-0001.sig>


More information about the libvir-list mailing list