From d436fafbde49d78e2e39a8a63c31747799531ca1 Mon Sep 17 00:00:00 2001 From: Manuel Buil Date: Fri, 26 Jan 2018 12:54:35 +0100 Subject: Fix problem to fetch the compute name of client VM When migrating to SNAPs a bug was introduced and could be seen in logs as: Error when waiting for classification rules: 'module' object has no attribute 'get_compute_client' The cause is that get_compute_client is not a global function anymore but a method inside a class. Besides, due to SNAPs, we need to fetch the compute name in another way Change-Id: I32007c83894281ba9514a121b7939b5bfe52f4bc Signed-off-by: Manuel Buil --- sfc/lib/odl_utils.py | 7 ++----- sfc/lib/openstack_utils.py | 11 +++++++++-- 2 files changed, 11 insertions(+), 7 deletions(-) (limited to 'sfc/lib') diff --git a/sfc/lib/odl_utils.py b/sfc/lib/odl_utils.py index 45937263..90b6b54f 100644 --- a/sfc/lib/odl_utils.py +++ b/sfc/lib/odl_utils.py @@ -102,17 +102,14 @@ def promised_rsps_in_computes(odl_ip, odl_port): @ft_utils.timethis def wait_for_classification_rules(ovs_logger, compute_nodes, odl_ip, odl_port, - timeout=200): + compute_client_name, timeout=200): ''' Check if the classification rules configured in ODL are implemented in OVS. We know by experience that this process might take a while ''' try: - # Find the compute where the client is - compute_client = os_sfc_utils.get_compute_client() - for compute_node in compute_nodes: - if compute_node.name in compute_client: + if compute_node.name in compute_client_name: compute = compute_node try: compute diff --git a/sfc/lib/openstack_utils.py b/sfc/lib/openstack_utils.py index 28cfebe7..7257b126 100644 --- a/sfc/lib/openstack_utils.py +++ b/sfc/lib/openstack_utils.py @@ -156,8 +156,15 @@ class OpenStackSFC: ''' Return the compute where the client sits ''' - compute = nova_utils.get_server(self.nova, server_name='client') - return compute + for creator in self.creators: + # We want to filter the vm creators + if hasattr(creator, 'get_vm_info'): + vm_info = creator.get_vm_info() + # We want to fetch only the client + if vm_info['name'] == 'client': + return vm_info['OS-EXT-SRV-ATTR:host'] + + raise Exception("There is no client VM!!") def assign_floating_ip(self, router, vm, vm_creator): ''' -- cgit 1.2.3-korg