| Message ID | 20200407155642.10716-2-ovoshcha@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Acceptance test: Extension of migration tests | expand |
On Tue, Apr 7, 2020 at 1:05 PM Oksana Vohchana <ovoshcha@redhat.com> wrote: > > In the migration test function _get_free_port works only for localhost, > but in the case to use migration through an RDMA we need to get a free port > on the configured network RDMA-interface. > This patch is the start for another migration option > > Signed-off-by: Oksana Vohchana <ovoshcha@redhat.com> > --- > tests/acceptance/migration.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/acceptance/migration.py b/tests/acceptance/migration.py > index a8367ca023..e4c39b85a1 100644 > --- a/tests/acceptance/migration.py > +++ b/tests/acceptance/migration.py > @@ -52,8 +52,8 @@ class Migration(Test): > source_vm.qmp('migrate', uri=src_uri) > self.assert_migration(source_vm, dest_vm) > > - def _get_free_port(self): > - port = network.find_free_port() > + def _get_free_port(self, address='localhost'): > + port = network.find_free_port(address=address) > if port is None: > self.cancel('Failed to find a free port') > return port > -- > 2.21.1 > > Reviewed-by: Willian Rampazzo <willianr@redhat.com>
diff --git a/tests/acceptance/migration.py b/tests/acceptance/migration.py index a8367ca023..e4c39b85a1 100644 --- a/tests/acceptance/migration.py +++ b/tests/acceptance/migration.py @@ -52,8 +52,8 @@ class Migration(Test): source_vm.qmp('migrate', uri=src_uri) self.assert_migration(source_vm, dest_vm) - def _get_free_port(self): - port = network.find_free_port() + def _get_free_port(self, address='localhost'): + port = network.find_free_port(address=address) if port is None: self.cancel('Failed to find a free port') return port
In the migration test function _get_free_port works only for localhost, but in the case to use migration through an RDMA we need to get a free port on the configured network RDMA-interface. This patch is the start for another migration option Signed-off-by: Oksana Vohchana <ovoshcha@redhat.com> --- tests/acceptance/migration.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)