commit 75b1f88cd2dd5eeb1fd817a2f3a291c2670f9c50
author Fabiano Rosas <farosas@suse.de> Wed Oct 11 15:46:04 2023 -0300
committer Peter Xu <peterx@redhat.com> Tue Jan 16 11:16:10 2024 +0800
tree e54e49f3ff68cbfa9984c6729b84e9a4694c85ec
parent 6f0771de903bb7623dc85bbf9f94f641979daaaa

tests/qtest: Re-enable multifd cancel test

We've found the source of flakiness in this test, so re-enable it.

Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Fabiano Rosas <farosas@suse.de>
Link: https://lore.kernel.org/r/20230606144551.24367-4-farosas@suse.de
[peterx: rebase to 2a61a6964c, to use migration_test_add()]
Signed-off-by: Peter Xu <peterx@redhat.com>

tests/qtest/migration-test.c

diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
index 21da140..d3066e1 100644
--- a/tests/qtest/migration-test.c
+++ b/tests/qtest/migration-test.c
@@ -3550,14 +3550,8 @@
     }
     migration_test_add("/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")) {
-        migration_test_add("/migration/multifd/tcp/plain/cancel",
-                           test_multifd_tcp_cancel);
-    }
+    migration_test_add("/migration/multifd/tcp/plain/cancel",
+                       test_multifd_tcp_cancel);
     migration_test_add("/migration/multifd/tcp/plain/zlib",
                        test_multifd_tcp_zlib);
 #ifdef CONFIG_ZSTD