From c75ba5028fd1abfca557351eaf0da086599a3be8 Mon Sep 17 00:00:00 2001 From: elajkat Date: Wed, 1 Dec 2021 15:54:28 +0100 Subject: [PATCH] Use admin_client to list admin's servers' ports FloatingIpTestCasesAdmin creates VMs as admin (to be sure to boot on availability_zone) so to list ports for the VM we have to use admin_client. Change-Id: I1da9ccc99b70355554ace3a20a237bb4f33381ab Closes-Bug: #1952806 --- .../scenario/admin/test_floatingip.py | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/neutron_tempest_plugin/scenario/admin/test_floatingip.py b/neutron_tempest_plugin/scenario/admin/test_floatingip.py index a08acc3d..d9abaf55 100644 --- a/neutron_tempest_plugin/scenario/admin/test_floatingip.py +++ b/neutron_tempest_plugin/scenario/admin/test_floatingip.py @@ -27,6 +27,14 @@ CONF = config.CONF class FloatingIpTestCasesAdmin(base.BaseTempestTestCase): credentials = ['primary', 'admin'] + @classmethod + def setup_clients(cls): + super(FloatingIpTestCasesAdmin, cls).setup_clients() + # admin_client set in BaseAdminNetworkTest but here we inherit from + # BaseNetworkTest + if not cls.admin_client: + cls.admin_client = cls.os_admin.network_client + @classmethod @utils.requires_ext(extension="router", service="network") def resource_setup(cls): @@ -75,7 +83,7 @@ class FloatingIpTestCasesAdmin(base.BaseTempestTestCase): waiters.wait_for_server_status( self.os_admin.servers_client, server['server']['id'], const.SERVER_STATUS_ACTIVE) - port = self.client.list_ports( + port = self.admin_client.list_ports( network_id=self.network['id'], device_id=server['server']['id'] )['ports'][0]