[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 15/18] tests: convert multifd migration tests to use common helpe
From: |
Daniel P . Berrangé |
Subject: |
[PATCH 15/18] tests: convert multifd migration tests to use common helper |
Date: |
Wed, 2 Mar 2022 17:49:29 +0000 |
Most of the multifd migration test logic is common with the rest of the
precopy tests, so it can use the helper without difficulty. The only
exception of the multifd cancellation test which tries to run multiple
migrations in a row.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
tests/qtest/migration-test.c | 75 +++++++++++++++++++-----------------
1 file changed, 39 insertions(+), 36 deletions(-)
diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
index 9896fcb134..7c69268aa8 100644
--- a/tests/qtest/migration-test.c
+++ b/tests/qtest/migration-test.c
@@ -1716,26 +1716,12 @@ static void test_migrate_auto_converge(void)
test_migrate_end(from, to, true);
}
-static void test_multifd_tcp(const char *method)
+static void *
+test_migration_precopy_tcp_multifd_start_common(QTestState *from,
+ QTestState *to,
+ const char *method)
{
- MigrateStart *args = migrate_start_new();
- QTestState *from, *to;
QDict *rsp;
- g_autofree char *uri = NULL;
-
- if (test_migrate_start(&from, &to, "defer", args)) {
- return;
- }
-
- /*
- * We want to pick a speed slow enough that the test completes
- * quickly, but that it doesn't complete precopy even on a slow
- * machine, so also set the downtime.
- */
- /* 1 ms should make it not converge*/
- migrate_set_parameter_int(from, "downtime-limit", 1);
- /* 1GB/s */
- migrate_set_parameter_int(from, "max-bandwidth", 1000000000);
migrate_set_parameter_int(from, "multifd-channels", 16);
migrate_set_parameter_int(to, "multifd-channels", 16);
@@ -1751,41 +1737,58 @@ static void test_multifd_tcp(const char *method)
" 'arguments': { 'uri': 'tcp:127.0.0.1:0' }}");
qobject_unref(rsp);
- /* Wait for the first serial output from the source */
- wait_for_serial("src_serial");
-
- uri = migrate_get_socket_address(to, "socket-address");
-
- migrate_qmp(from, uri, "{}");
+ return NULL;
+}
- wait_for_migration_pass(from);
+static void *
+test_migration_precopy_tcp_multifd_start(QTestState *from,
+ QTestState *to)
+{
+ return test_migration_precopy_tcp_multifd_start_common(from, to, "none");
+}
- migrate_set_parameter_int(from, "downtime-limit", CONVERGE_DOWNTIME);
+static void *
+test_migration_precopy_tcp_multifd_zlib_start(QTestState *from,
+ QTestState *to)
+{
+ return test_migration_precopy_tcp_multifd_start_common(from, to, "zlib");
+}
- if (!got_stop) {
- qtest_qmp_eventwait(from, "STOP");
- }
- qtest_qmp_eventwait(to, "RESUME");
+#ifdef CONFIG_ZSTD
+static void *
+test_migration_precopy_tcp_multifd_zstd_start(QTestState *from,
+ QTestState *to)
+{
+ return test_migration_precopy_tcp_multifd_start_common(from, to, "zstd");
+}
+#endif /* CONFIG_ZSTD */
- wait_for_serial("dest_serial");
- wait_for_migration_complete(from);
- test_migrate_end(from, to, true);
+static void test_multifd_tcp_common(TestMigrateStartHook start_hook)
+{
+ test_precopy_common("defer",
+ NULL, /* connect_uri */
+ start_hook,
+ NULL, /* finish_hook */
+ false, /* expect_fail */
+ false, /* dst_quit */
+ 1, /* iterations */
+ false /* dirty_ring */);
}
static void test_multifd_tcp_none(void)
{
- test_multifd_tcp("none");
+ test_multifd_tcp_common(test_migration_precopy_tcp_multifd_start);
}
static void test_multifd_tcp_zlib(void)
{
- test_multifd_tcp("zlib");
+ test_multifd_tcp_common(test_migration_precopy_tcp_multifd_zlib_start);
}
#ifdef CONFIG_ZSTD
static void test_multifd_tcp_zstd(void)
{
- test_multifd_tcp("zstd");
+ test_multifd_tcp_common(test_migration_precopy_tcp_multifd_zstd_start);
}
#endif
--
2.34.1
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, (continued)
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Daniel P . Berrangé, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Marc-André Lureau, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Daniel P . Berrangé, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Marc-André Lureau, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Daniel P . Berrangé, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Marc-André Lureau, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Thomas Huth, 2022/03/10
- Re: [PATCH 04/18] tests: print newline after QMP response in qtest logs, Dr. David Alan Gilbert, 2022/03/10
[PATCH 14/18] tests: convert XBZRLE migration test to use common helper, Daniel P . Berrangé, 2022/03/02
[PATCH 15/18] tests: convert multifd migration tests to use common helper,
Daniel P . Berrangé <=
[PATCH 16/18] tests: add multifd migration tests of TLS with PSK credentials, Daniel P . Berrangé, 2022/03/02
[PATCH 17/18] tests: add multifd migration tests of TLS with x509 credentials, Daniel P . Berrangé, 2022/03/02
[PATCH 18/18] tests: ensure migration status isn't reported as failed, Daniel P . Berrangé, 2022/03/02