x/vmware-nsx
Code Issues Proposed changes

NSXv: Fix pool logging calls

Browse Source 
LBaaSv2 pool is and object and not a dict.
Log calls should use object attribute to get pool id.

Change-Id: I1b5650b8bb655d3b0a92681138f79169de388a62
This commit is contained in:
Kobi Samoray 2017-01-25 19:52:04 +02:00
parent 34e165181a
commit 9e4bc9296f
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
vmware_nsx/services/lbaas/nsx_v/v2/pool_mgr.py
View File

@ -77,7 +77,7 @@ class EdgePoolManager(base_mgr.EdgeLoadbalancerBaseManager):
except nsxv_exc.VcnsApiException:
with excutils.save_and_reraise_exception():
self.lbv2_driver.pool.failed_completion(context, pool)
LOG.error(_LE('Failed to create pool %s'), pool['id'])
LOG.error(_LE('Failed to create pool %s'), pool.id)
@log_helpers.log_method_call
def update(self, context, old_pool, new_pool):
@ -113,7 +113,7 @@ class EdgePoolManager(base_mgr.EdgeLoadbalancerBaseManager):
except nsxv_exc.VcnsApiException:
with excutils.save_and_reraise_exception():
self.lbv2_driver.pool.failed_completion(context, new_pool)
LOG.error(_LE('Failed to update pool %s'), new_pool['id'])
LOG.error(_LE('Failed to update pool %s'), new_pool.id)
@log_helpers.log_method_call
def delete(self, context, pool):
@ -147,4 +147,4 @@ class EdgePoolManager(base_mgr.EdgeLoadbalancerBaseManager):
context.session, lb_id, pool.id)
except nsxv_exc.VcnsApiException:
self.lbv2_driver.pool.failed_completion(context, pool)
LOG.error(_LE('Failed to delete pool %s'), pool['id'])
LOG.error(_LE('Failed to delete pool %s'), pool.id)