Skip to content

Commit e566eb7

Browse files
authored
Merge pull request #865 from 08volt/targetpool-leak
Fix: Target Pool are leaked when Forwarding Rule fails deletion
2 parents c19c597 + 9a8c261 commit e566eb7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

providers/gce/gce_loadbalancer.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ func (g *Cloud) GetLoadBalancer(ctx context.Context, clusterName string, svc *v1
116116
}
117117
// Checking for finalizer is more accurate because controller restart could happen in the middle of resource
118118
// deletion. So even though forwarding rule was deleted, cleanup might not have been complete.
119-
if hasFinalizer(svc, ILBFinalizerV1) {
119+
if hasFinalizer(svc, ILBFinalizerV1) || hasFinalizer(svc, NetLBFinalizerV1) {
120120
return &v1.LoadBalancerStatus{}, true, nil
121121
}
122122
return nil, false, ignoreNotFound(err)

0 commit comments

Comments
 (0)