[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: |
Fabiano Rosas |
Subject: |
Re: [PATCH 1/4] Revert "migration: modify test_multifd_tcp_none() to use new QAPI syntax" |
Date: |
Thu, 11 Apr 2024 16:41:16 -0300 |
Peter Xu <peterx@redhat.com> writes:
> On Thu, Apr 11, 2024 at 11:31:08PM +0530, Het Gala wrote:
>> I just wanted to highlight couple of pointers:
>> 1. though we are using 'channels' in the precopy tests for 'migrate' QAPI,
>> we
>> use the old uri for 'migrate-incoming' QAPI.
>> 2. We do not cover other 'channels' abi, only have tcp path tested.
>>
>> So, the TO-DOs could be:
>> 1. Omit the 4th patch here, which introduced postcopy qtests with 'channels'
>> interface OR have 'channels' interface with other than tcp transport
>> (file, exec, vsock, etc) so as to cover different code paths.
>> 2. Extend channels interface to migrate-incoming QAPI for precopy qtests
>
> You can see whether Fabiano has anything to say, but what you proposed
> looks good to me.
Ok, so what about we convert some of the 'plain' tests into channels to
cover all transports?
- tcp: test_multifd_tcp_none (this one we already did)
- file: test_precopy_file
- unix: test_precopy_unix_plain
- exec: test_analyze_script
- fd: test_migrate_precopy_fd_socket
Those^, plus the validate_uri that's already in next should cover
everything.
We don't need to do this at once, by the way.
Moreover:
- leave all test strings untouched to preserve bisecting;
- let's not bother adding "channels" and "uri" to the test string
anymore. The channels API should be taken for granted at this point, I
don't expect we start hitting bugs that will require us to run either
foo/uri/plain or foo/channels/plain, so there's not much point in
making the distinction.
- [PATCH 4/4] tests/qtest/migration: Add postcopy migration qtests to use 'channels' argument instead of uri, (continued)
- [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, 2024/04/10
- 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 <=
- 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