[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/3] qtest: migration: Introduce 'connect_channels' in Migrat
From: |
Peter Xu |
Subject: |
Re: [PATCH 2/3] qtest: migration: Introduce 'connect_channels' in MigrateCommon struct |
Date: |
Tue, 20 Feb 2024 14:04:57 +0800 |
On Fri, Feb 16, 2024 at 09:06:23AM +0000, Het Gala wrote:
> migration QAPIs can now work with either 'channels' or 'uri' as their
> argument.
>
> Signed-off-by: Het Gala <het.gala@nutanix.com>
> ---
> tests/qtest/migration-test.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
> index e7f2719dcf..0bc69b1943 100644
> --- a/tests/qtest/migration-test.c
> +++ b/tests/qtest/migration-test.c
> @@ -704,6 +704,13 @@ typedef struct {
> */
> const char *connect_uri;
>
> + /*
> + * Optional: list of migration stream channels, each connected
> + * to a dst QEMU. It can be used instead of URI to carry out
> + * the same task as listen_uri or connect_uri.
> + */
> + MigrationChannelList *connect_channels;
> +
> /* Optional: callback to run at start to set migration parameters */
> TestMigrateStartHook start_hook;
> /* Optional: callback to run at finish to cleanup */
Please squash this patch into the follow up patch that uses it. Thanks,
--
Peter Xu
[PATCH 3/3] qtest: migration: Add negative validation test for 'uri' and 'channels' both set, Het Gala, 2024/02/16