DevFW-CICD/ingress-nginx-helm
14
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Ignore instance groups in use

Browse source
This commit is contained in:
Nick Sardo 2017-08-02 15:39:25 -07:00
parent 504bf929fd
commit 27e42e9067
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
controllers/gce/instances/instances.go
View file

@ -130,11 +130,15 @@ func (i *Instances) DeleteInstanceGroup(name string) error {
} }
for _, zone := range zones { for _, zone := range zones {
if err := i.cloud.DeleteInstanceGroup(name, zone); err != nil { if err := i.cloud.DeleteInstanceGroup(name, zone); err != nil {
if !utils.IsHTTPErrorCode(err, http.StatusNotFound) { if utils.IsNotFoundError(err) {
glog.V(3).Infof("Instance group %v in zone %v did not exist", name, zone)
} else if utils.IsInUsedByError(err) {
glog.V(3).Infof("Could not delete instance group %v in zone %v because it's still in use. Ignoring: %v", name, zone, err)
} else {
errs = append(errs, err) errs = append(errs, err)
} }
} else { } else {
glog.Infof("Deleted instance group %v in zone %v", name, zone) glog.V(3).Infof("Deleted instance group %v in zone %v", name, zone)
} }
} }
if len(errs) == 0 { if len(errs) == 0 {