this is v3. Please increment the version in the patch subject.
You could also send a simple "ping" to the specific patch, instead of
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 <ovosh...@redhat.com>
v1 of this patch has already received reviewers tags
please collect them and keep them when you resend the same patch:
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Tested-by: Wainer dos Santos Moschetta <waine...@redhat.com>
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
@@ -70,8 +70,8 @@ class Migration(Test):
@skipUnless(find_command('nc', default=False), "'nc' command not found")
- The test works for both netcat-traditional and netcat-openbsd packages
+ """The test works for both netcat-traditional and netcat-openbsd
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?