Fix: Target Pool are leaked when Forwarding Rule is deleted

Adding finalizer v1 check when checking if NetLB exists. If FR was deleted, the controller was thinking the LB did not exist anymore and skip cleaning up the Target Pool

Author: 08volt

providers/gce/gce_loadbalancer.go

@@ -116,7 +116,7 @@ func (g *Cloud) GetLoadBalancer(ctx context.Context, clusterName string, svc *v1
 	}
 	// Checking for finalizer is more accurate because controller restart could happen in the middle of resource
 	// deletion. So even though forwarding rule was deleted, cleanup might not have been complete.
-	if hasFinalizer(svc, ILBFinalizerV1) {
+	if hasFinalizer(svc, ILBFinalizerV1) || hasFinalizer(svc, NetLBFinalizerV1) {
 		return &v1.LoadBalancerStatus{}, true, nil
 	}
 	return nil, false, ignoreNotFound(err)