From 785a775777cc6d4fab14415e0c776f91d8550221 Mon Sep 17 00:00:00 2001 From: Angus Lees Date: Tue, 2 Jun 2015 14:00:25 +1000 Subject: [PATCH] Actually delete LBaaS monitors after disassociating Partially addresses issue #8352 --- pkg/cloudprovider/openstack/openstack.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/cloudprovider/openstack/openstack.go b/pkg/cloudprovider/openstack/openstack.go index 0ef02cb72df..8707434052a 100644 --- a/pkg/cloudprovider/openstack/openstack.go +++ b/pkg/cloudprovider/openstack/openstack.go @@ -691,8 +691,8 @@ func (lb *LoadBalancer) EnsureTCPLoadBalancerDeleted(name, region string) error if poolExists { for _, monId := range pool.MonitorIDs { - // TODO(#8352): Delete the monitor, don't just disassociate it. pools.DisassociateMonitor(lb.network, pool.ID, monId) + monitors.Delete(monId) } pools.Delete(lb.network, pool.ID) }