From 89c0205f5d3d0575abcd0804e25a54ef1eace336 Mon Sep 17 00:00:00 2001 From: Matthew Greene Date: Fri, 6 May 2016 09:25:29 -0600 Subject: [PATCH] Issues: Fixes #96 Problem: Unable to pass traffic through VE deployed in guest tenant Analysis: This deployment of a VE inside the tenant has never been tested. The code was attempting to reference a nested dictionary key that is not populated in 'global routed mode'. The necessary value is contained in the parent dictionary. Tests: Standard "overcloud" deployment. Run traffic test from lbaasv2-driver repo. (We're working on backporting tests to lbaasv1). --- agent/f5/oslbaasv1agent/drivers/bigip/vips.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/agent/f5/oslbaasv1agent/drivers/bigip/vips.py b/agent/f5/oslbaasv1agent/drivers/bigip/vips.py index fc9ea70..aab71a6 100644 --- a/agent/f5/oslbaasv1agent/drivers/bigip/vips.py +++ b/agent/f5/oslbaasv1agent/drivers/bigip/vips.py @@ -72,7 +72,7 @@ def assure_bigip_create_vip(self, bigip, service, traffic_group): just_added_vip: self._update_bigip_vip(bigip, service) if self.l3_binding: - self.l3_binding.bind_address(subnet_id=vip['subnet']['id'], + self.l3_binding.bind_address(subnet_id=vip['subnet_id'], ip_address=ip_address) def assure_bigip_delete_vip(self, bigip, service): @@ -98,7 +98,7 @@ def assure_bigip_delete_vip(self, bigip, service): vip['id'], folder=vip['tenant_id']) if self.l3_binding: - self.l3_binding.unbind_address(subnet_id=vip['subnet']['id'], + self.l3_binding.unbind_address(subnet_id=vip['subnet_id'], ip_address=vip['address']) def _create_bigip_vip(self, bigip, service, vip_info):