[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] tests/qtest: Re-enable multifd cancel test
From: |
Juan Quintela |
Subject: |
Re: [PATCH 3/3] tests/qtest: Re-enable multifd cancel test |
Date: |
Wed, 07 Jun 2023 10:27:15 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Fabiano Rosas <farosas@suse.de> wrote:
> We've found the source of flakiness in this test, so re-enable it.
>
> Signed-off-by: Fabiano Rosas <farosas@suse.de>
> ---
> tests/qtest/migration-test.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
> index b0c355bbd9..800ad23b75 100644
> --- a/tests/qtest/migration-test.c
> +++ b/tests/qtest/migration-test.c
> @@ -2778,14 +2778,8 @@ int main(int argc, char **argv)
> }
> qtest_add_func("/migration/multifd/tcp/plain/none",
> test_multifd_tcp_none);
> - /*
> - * This test is flaky and sometimes fails in CI and otherwise:
> - * don't run unless user opts in via environment variable.
> - */
> - if (getenv("QEMU_TEST_FLAKY_TESTS")) {
> - qtest_add_func("/migration/multifd/tcp/plain/cancel",
> - test_multifd_tcp_cancel);
> - }
> + qtest_add_func("/migration/multifd/tcp/plain/cancel",
> + test_multifd_tcp_cancel);
> qtest_add_func("/migration/multifd/tcp/plain/zlib",
> test_multifd_tcp_zlib);
> #ifdef CONFIG_ZSTD
Reviewed-by: Juan Quintela <quintela@redhat.com>
There was another failure with migration test that I will post during
the rest of the day. It needs both to get it right.
Later, Juan.
- Re: [PATCH 1/3] migration/multifd: Rename threadinfo.c functions, (continued)
[PATCH 3/3] tests/qtest: Re-enable multifd cancel test, Fabiano Rosas, 2023/06/06
- Re: [PATCH 3/3] tests/qtest: Re-enable multifd cancel test,
Juan Quintela <=