Fixes bug 1153132

Change-Id: I66d8bb16cbecc5072d20d2a9b8db8ebf598bf291
This commit is contained in:
Gary Kotton 2013-03-10 07:35:29 +00:00
parent 1001685a9f
commit cad2caf1d5
2 changed files with 2 additions and 3 deletions

View File

@ -267,7 +267,6 @@ class LoadBalancerPlugin(loadbalancer_db.LoadBalancerPluginDb):
vip = self.get_vip(context, id) vip = self.get_vip(context, id)
super(LoadBalancerPlugin, self).delete_vip(context, id) super(LoadBalancerPlugin, self).delete_vip(context, id)
self.agent_rpc.destroy_pool(context, vip['pool_id']) self.agent_rpc.destroy_pool(context, vip['pool_id'])
pass
def create_pool(self, context, pool): def create_pool(self, context, pool):
p = super(LoadBalancerPlugin, self).create_pool(context, pool) p = super(LoadBalancerPlugin, self).create_pool(context, pool)

View File

@ -250,13 +250,13 @@ class TestLoadBalancerPluginNotificationWrapper(TestLoadBalancerPluginBase):
constants.PENDING_UPDATE constants.PENDING_UPDATE
) )
def t2est_delete_vip(self): def test_delete_vip(self):
with self.subnet() as subnet: with self.subnet() as subnet:
with self.pool(subnet=subnet) as pool: with self.pool(subnet=subnet) as pool:
with self.vip(pool=pool, subnet=subnet, no_delete=True) as vip: with self.vip(pool=pool, subnet=subnet, no_delete=True) as vip:
self.mock_api.reset_mock() self.mock_api.reset_mock()
ctx = context.get_admin_context() ctx = context.get_admin_context()
self.plugin_instance.delete_vip(context, vip['vip']['id']) self.plugin_instance.delete_vip(ctx, vip['vip']['id'])
self.mock_api.destroy_pool.assert_called_once_with( self.mock_api.destroy_pool.assert_called_once_with(
mock.ANY, mock.ANY,
vip['vip']['pool_id'] vip['vip']['pool_id']