Fix panic in GCE loadbalancer library

This commit is contained in:
Marek Grabowski 2017-11-24 13:25:38 +00:00
parent 58fca39de3
commit 94b6a1fb70

View File

@ -344,7 +344,10 @@ func (gce *GCECloud) ensureExternalLoadBalancerDeleted(clusterName, clusterID st
glog.Infof("ensureExternalLoadBalancerDeleted(%s): Failed to retrieve health check:%v.", lbRefStr, err) glog.Infof("ensureExternalLoadBalancerDeleted(%s): Failed to retrieve health check:%v.", lbRefStr, err)
return err return err
} }
hcNames = append(hcNames, hcToDelete.Name) // If we got 'StatusNotFound' LB was already deleted and it's safe to ignore.
if err == nil {
hcNames = append(hcNames, hcToDelete.Name)
}
} else { } else {
// EnsureLoadBalancerDeleted() could be triggered by changing service from // EnsureLoadBalancerDeleted() could be triggered by changing service from
// LoadBalancer type to others. In this case we have no idea whether it was // LoadBalancer type to others. In this case we have no idea whether it was