Re: [PATCH] tests: Disable dbus-vmstate-test

[Laurent Vivier <lvivier@redhat.com>]

On 10/03/2020 16:21, Peter Maydell wrote:
> The dbus-vmstate-test has been failing in some Patchew configs
> since about the 6th March:
> 
>   dbus-daemon[9321]: Could not get password database information for UID of current process: User "???" unknown or no memory to allocate password entry
> 
>   **
>   ERROR:/tmp/qemu-test/src/tests/qtest/dbus-vmstate-test.c:114:get_connection: assertion failed (err == NULL): The connection is closed (g-io-error-quark, 18)
>   cleaning up pid 9321
>   ERROR - Bail out! ERROR:/tmp/qemu-test/src/tests/qtest/dbus-vmstate-test.c:114:get_connection: assertion failed (err == NULL): The connection is closed (g-io-error-quark, 18)
>   make: *** [/tmp/qemu-test/src/tests/Makefile.include:632: check-qtest-x86_64] Error 1
>   make: *** Waiting for unfinished jobs....
> 
> It's not clear why this is happening (perhaps a recently revealed
> race condition or a change in the patchew build environment?).
> 
> For the moment, disable this test so that patchew test runs are
> useful and don't email the list with spurious failure mails.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  tests/qtest/Makefile.include | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/qtest/Makefile.include b/tests/qtest/Makefile.include
> index 383b0ab2171..5115f7897db 100644
> --- a/tests/qtest/Makefile.include
> +++ b/tests/qtest/Makefile.include
> @@ -18,7 +18,8 @@ check-qtest-pci-$(CONFIG_IVSHMEM_DEVICE) += ivshmem-test
>  DBUS_DAEMON := $(shell which dbus-daemon 2>/dev/null)
>  ifneq ($(GDBUS_CODEGEN),)
>  ifneq ($(DBUS_DAEMON),)
> -check-qtest-pci-$(CONFIG_GIO) += dbus-vmstate-test
> +# Temporarily disabled due to Patchew failures:
> +#check-qtest-pci-$(CONFIG_GIO) += dbus-vmstate-test
>  endif
>  endif
>  

Adding Marc-André.

The MAINTAINERS file has not been updated when the file has been moved
from tests/ to tests/qtest

Thanks,
Laurent