| Message ID | 20240619223046.1798968-11-peterx@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | migration: New postcopy state, and some cleanups | expand |
Peter Xu <peterx@redhat.com> writes: > Making sure the postcopy-recover-setup status is present in the postcopy > failure unit test. Note that it only applies to src QEMU not dest. > > This also introduces the tiny but helpful migration_event_wait() helper. Not anymore. I'll drop this line. Reviewed-by: Fabiano Rosas <farosas@suse.de>
diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c index c015e801ac..de81e28088 100644 --- a/tests/qtest/migration-test.c +++ b/tests/qtest/migration-test.c @@ -1412,6 +1412,12 @@ static void postcopy_recover_fail(QTestState *from, QTestState *to) migrate_recover(to, "fd:fd-mig"); migrate_qmp(from, to, "fd:fd-mig", NULL, "{'resume': true}"); + /* + * Source QEMU has an extra RECOVER_SETUP phase, dest doesn't have it. + * Make sure it appears along the way. + */ + migration_event_wait(from, "postcopy-recover-setup"); + /* * Make sure both QEMU instances will go into RECOVER stage, then test * kicking them out using migrate-pause.
Making sure the postcopy-recover-setup status is present in the postcopy failure unit test. Note that it only applies to src QEMU not dest. This also introduces the tiny but helpful migration_event_wait() helper. Signed-off-by: Peter Xu <peterx@redhat.com> --- tests/qtest/migration-test.c | 6 ++++++ 1 file changed, 6 insertions(+)