gcp: destroy instance groups

Author: abhinavdahiya

pkg/destroy/bootstrap/bootstrap.go

@@ -54,6 +54,7 @@ func Destroy(dir string) (err error) {
 	switch platform {
 	case gcp.Name:
 		// First remove the bootstrap from LB target and its instance so that bootstrap module is cleanly destroyed.
+		fmt.Println("RUNNING TF APPLY GCP_BOOTSTRAP_ENABLED FALSE")
 		_, err = terraform.Apply(tempDir, platform, append(extraArgs, "-var=gcp_bootstrap_enabled=false")...)
 		if err != nil {
 			return errors.Wrap(err, "Terraform apply")

pkg/destroy/gcp/compute.go

@@ -111,6 +111,29 @@ func (o *ClusterUninstaller) deleteInstanceGroup(ig nameAndZone) error {
 	return nil
 }
 
+// destroyInstanceGroups searches for instance groups across all zones that have a name that starts with
+// the infra ID prefix, and then deletes each instance found.
+func (o *ClusterUninstaller) destroyInstanceGroups() error {
+	groups, err := o.listInstanceGroupsWithFilter(o.clusterIDFilter())
+	if err != nil {
+		return err
+	}
+	errs := []error{}
+	found := make([]string, 0, len(groups))
+	for _, group := range groups {
+		found = append(found, fmt.Sprintf("%s/%s", group.zone, group.name))
+		err := o.deleteInstanceGroup(group)
+		if err != nil {
+			errs = append(errs, err)
+		}
+	}
+	deletedItems := o.setPendingItems("computeinstancegroup", found)
+	for _, item := range deletedItems {
+		o.Logger.Infof("Deleted instance group %s", item)
+	}
+	return aggregateError(errs, len(found))
+}
+
 func (o *ClusterUninstaller) listComputeInstances() ([]nameAndZone, error) {
 	o.Logger.Debugf("Listing compute instances")
 	result := []nameAndZone{}

pkg/destroy/gcp/gcp.go

@@ -134,6 +134,7 @@ func (o *ClusterUninstaller) destroyCluster() (bool, error) {
 		{name: "Firewalls", destroy: o.destroyFirewalls},
 		{name: "Addresses", destroy: o.destroyAddresses},
 		{name: "Target Pools", destroy: o.destroyTargetPools},
+		{name: "Compute instance groups", destroy: o.destroyInstanceGroups},
 		{name: "Forwarding rules", destroy: o.destroyForwardingRules},
 		{name: "Backend services", destroy: o.destroyBackendServices},
 		{name: "Health checks", destroy: o.destroyHealthChecks},