Re: [PATCH-for-5.0 v3] Acceptance test: Fix to EXEC migration

[Philippe Mathieu-Daudé]

Hi Oksana,

v2 was 
https://www.mail-archive.com/address@hidden/msg682899.html, so 
this is v3. Please increment the version in the patch subject.

You could also send a simple "ping" to the specific patch, instead of 
resending it.

On 3/25/20 12:31 PM, Oksana Vohchana wrote:
> The exec migration test isn't run a whole test scenario.
> This patch fixes it
>
> Signed-off-by: Oksana Vohchana <address@hidden>

v1 of this patch has already received reviewers tags 
(https://www.mail-archive.com/address@hidden/msg679629.html), 
please collect them and keep them when you resend the same patch:

Fixes: 2e768cb682bf
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Wainer dos Santos Moschetta <address@hidden>

> ---
>   tests/acceptance/migration.py | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tests/acceptance/migration.py b/tests/acceptance/migration.py
> index a8367ca023..0365289cda 100644
> --- a/tests/acceptance/migration.py
> +++ b/tests/acceptance/migration.py
> @@ -70,8 +70,8 @@ class Migration(Test):
>   
>       @skipUnless(find_command('nc', default=False), "'nc' command not found")
>       def test_migration_with_exec(self):
> -        """
> -        The test works for both netcat-traditional and netcat-openbsd packages
> -        """
> +        """The test works for both netcat-traditional and netcat-openbsd 
> packages."""

Btw why are you changing the comment style?

>           free_port = self._get_free_port()
>           dest_uri = 'exec:nc -l localhost %u' % free_port
> +        src_uri = 'exec:nc localhost %u' % free_port
> +        self.do_migrate(dest_uri, src_uri)

Alex, if there is no Python testing pullreq, can you take this patch via 
your testing tree?

Thanks,

Phil.