[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 7/7] tests/qtest/vhost-user-test: enable the reconnect tes
From: |
Raphael Norwitz |
Subject: |
Re: [PATCH v4 7/7] tests/qtest/vhost-user-test: enable the reconnect tests |
Date: |
Tue, 8 Sep 2020 23:06:47 -0400 |
This works for me, and looks good, but I figure those who added the
check should confirm that these tests are reliable now.
Marc-Andre - thoughts?
On Fri, Sep 4, 2020 at 5:36 AM Dima Stepanov <dimastep@yandex-team.ru> wrote:
>
> For now a QTEST_VHOST_USER_FIXME environment variable is used to
> separate reconnect tests for the vhost-user-net device. Looks like the
> reconnect functionality is pretty stable, so this separation is
> deprecated.
> Remove it and enable these tests for the default run.
>
> Signed-off-by: Dima Stepanov <dimastep@yandex-team.ru>
> ---
> tests/qtest/vhost-user-test.c | 25 +++++++++++--------------
> 1 file changed, 11 insertions(+), 14 deletions(-)
>
> diff --git a/tests/qtest/vhost-user-test.c b/tests/qtest/vhost-user-test.c
> index 4b715d3..4b96312 100644
> --- a/tests/qtest/vhost-user-test.c
> +++ b/tests/qtest/vhost-user-test.c
> @@ -1140,20 +1140,17 @@ static void register_vhost_user_test(void)
> "virtio-net",
> test_migrate, &opts);
>
> - /* keeps failing on build-system since Aug 15 2017 */
> - if (getenv("QTEST_VHOST_USER_FIXME")) {
> - opts.before = vhost_user_test_setup_reconnect;
> - qos_add_test("vhost-user/reconnect", "virtio-net",
> - test_reconnect, &opts);
> -
> - opts.before = vhost_user_test_setup_connect_fail;
> - qos_add_test("vhost-user/connect-fail", "virtio-net",
> - test_vhost_user_started, &opts);
> -
> - opts.before = vhost_user_test_setup_flags_mismatch;
> - qos_add_test("vhost-user/flags-mismatch", "virtio-net",
> - test_vhost_user_started, &opts);
> - }
> + opts.before = vhost_user_test_setup_reconnect;
> + qos_add_test("vhost-user/reconnect", "virtio-net",
> + test_reconnect, &opts);
> +
> + opts.before = vhost_user_test_setup_connect_fail;
> + qos_add_test("vhost-user/connect-fail", "virtio-net",
> + test_vhost_user_started, &opts);
> +
> + opts.before = vhost_user_test_setup_flags_mismatch;
> + qos_add_test("vhost-user/flags-mismatch", "virtio-net",
> + test_vhost_user_started, &opts);
>
> opts.before = vhost_user_test_setup_multiqueue;
> opts.edge.extra_device_opts = "mq=on";
> --
> 2.7.4
>
>
- Re: [PATCH v4 3/7] tests/qtest/vhost-user-test: prepare the tests for adding new dev class, (continued)