[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use
From: |
Peter Xu |
Subject: |
Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax" |
Date: |
Wed, 10 Apr 2024 10:53:04 -0400 |
On Wed, Apr 10, 2024 at 10:04:33AM -0300, Fabiano Rosas wrote:
> Het Gala <het.gala@nutanix.com> writes:
>
> > This reverts commit 8e3766eefbb4036cbc280c1f1a0d28537929f7fb
> >
> > After addition of 'channels' as the starting argument of new QAPI
> > syntax inside postcopy test, even if the user entered the old QAPI
> > syntax, test used the new syntax.
> > It was a temporary patch added to have some presence of the new syntax
> > since the migration qtest framework lacked any logic for introducing
> > 'channels' argument.
>
> That wasn't clear to me when we merged that. Was that really the case?
Yeah these look all a bit confusing..
I'm wondering whether do we need the new interface to cover both precopy
and postcopy, or one would suffice?
Both should share the same interface. I think it means if we covered the
channels interface in precopy, then perhaps we don't need to test anywhere
else, as we got the code paths all covered.
We actually do the same already for all kinds of channels for postcopy,
where we stick with either tcp/unix but don't cover the rest.
--
Peter Xu
- [PATCH 3/4] tests/qtest/migration: Add channels parameter in migrate_incoming_qmp, (continued)
- [PATCH 3/4] tests/qtest/migration: Add channels parameter in migrate_incoming_qmp, Het Gala, 2024/04/10
- [PATCH 2/4] tests/qtest/migration: Replace 'migrate-incoming' qtest_qmp_assert_success with migrate_incoming_qmp, Het Gala, 2024/04/10
- [PATCH 4/4] tests/qtest/migration: Add postcopy migration qtests to use 'channels' argument instead of uri, Het Gala, 2024/04/10
- [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Het Gala, 2024/04/10
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Fabiano Rosas, 2024/04/10
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax",
Peter Xu <=
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Het Gala, 2024/04/11
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Peter Xu, 2024/04/11
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Het Gala, 2024/04/11
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Peter Xu, 2024/04/11
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Fabiano Rosas, 2024/04/11
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Peter Xu, 2024/04/12
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Fabiano Rosas, 2024/04/12
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Peter Xu, 2024/04/12
- Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax", Het Gala, 2024/04/11