| Message ID | 20200214145235.4378-3-ovoshcha@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Extension of migration tests | expand |
On 2/14/20 3:52 PM, Oksana Vohchana wrote: > Improves EXEC migration to run whole test stage > > Signed-off-by: Oksana Vohchana <ovoshcha@redhat.com> Fixes: 2e768cb682bf > --- > tests/acceptance/migration.py | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tests/acceptance/migration.py b/tests/acceptance/migration.py > index e4c39b85a1..8209dcf71d 100644 > --- a/tests/acceptance/migration.py > +++ b/tests/acceptance/migration.py > @@ -75,3 +75,5 @@ class Migration(Test): > """ > 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) > I was very surprised because commit 2e768cb682bf is not the content I reviewed. Apparently what happened here is you sent 2 different series with the same subject... I got confused while applying I selected the incorrect id: $ pwclient list 'Acceptance test: provides to use different transport for migration' ID State Name -- ----- ---- 11348377 New Acceptance test: provides to use different transport for migration 11348719 New Acceptance test: provides to use different transport for migration 11355797 New [v2] Acceptance test: provides to use different transport for migration 11358447 New [v2,REPOST] Acceptance test: provides to use different transport for migration 11362397 New [v3,1/2] Acceptance test: provides to use different transport for migration 11362399 New [v3,2/2] Acceptance test: provides to use different transport for migration 11362479 New [v3,1/2] Acceptance test: provides to use different transport for migration 11362485 New [v3,2/2] Acceptance test: provides to use different transport for migration I am sorry I missed that. Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
On 2/14/20 12:52 PM, Oksana Vohchana wrote: > Improves EXEC migration to run whole test stage > > Signed-off-by: Oksana Vohchana <ovoshcha@redhat.com> > --- > tests/acceptance/migration.py | 2 ++ > 1 file changed, 2 insertions(+) Indeed, with this changes the migration is triggered. Tested-by: Wainer dos Santos Moschetta <wainersm@redhat.com> > > diff --git a/tests/acceptance/migration.py b/tests/acceptance/migration.py > index e4c39b85a1..8209dcf71d 100644 > --- a/tests/acceptance/migration.py > +++ b/tests/acceptance/migration.py > @@ -75,3 +75,5 @@ class Migration(Test): > """ > 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)
diff --git a/tests/acceptance/migration.py b/tests/acceptance/migration.py index e4c39b85a1..8209dcf71d 100644 --- a/tests/acceptance/migration.py +++ b/tests/acceptance/migration.py @@ -75,3 +75,5 @@ class Migration(Test): """ 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)
Improves EXEC migration to run whole test stage Signed-off-by: Oksana Vohchana <ovoshcha@redhat.com> --- tests/acceptance/migration.py | 2 ++ 1 file changed, 2 insertions(+)