Skip to content

Commit 93388a7

Browse files
committed
feat: adding firewall deletion
1 parent 2e95ac4 commit 93388a7

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

gcp/compute_firewalls.go

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,7 @@ func (c *ComputeFirewalls) List(refreshCache bool) []string {
6363
}
6464

6565
for _, firewall := range firewallList.Items {
66-
67-
c.resourceMap.Store(firewall.Name, firewall.Name)
66+
c.resourceMap.Store(firewall.Name, nil)
6867
}
6968
return c.ToSlice()
7069
}
@@ -85,7 +84,6 @@ func (c *ComputeFirewalls) Remove() error {
8584

8685
c.resourceMap.Range(func(key, value interface{}) bool {
8786
firewallID := key.(string)
88-
zone := value.(DefaultResourceProperties).zone
8987

9088
// Parallel firewall deletion
9189
errs.Go(func() error {
@@ -97,9 +95,9 @@ func (c *ComputeFirewalls) Remove() error {
9795
var opStatus string
9896
seconds := 0
9997
for opStatus != "DONE" {
100-
log.Printf("[Info] Resource currently being deleted %v [type: %v project: %v zone: %v] (%v seconds)", firewallID, c.Name(), c.base.config.Project, zone, seconds)
98+
log.Printf("[Info] Resource currently being deleted %v [type: %v project: %v] (%v seconds)", firewallID, c.Name(), c.base.config.Project, seconds)
10199

102-
operationCall := c.serviceClient.ZoneOperations.Get(c.base.config.Project, zone, operation.Name)
100+
operationCall := c.serviceClient.GlobalOperations.Get(c.base.config.Project, operation.Name)
103101
checkOpp, err := operationCall.Do()
104102
if err != nil {
105103
return err
@@ -109,12 +107,12 @@ func (c *ComputeFirewalls) Remove() error {
109107
time.Sleep(time.Duration(c.base.config.PollTime) * time.Second)
110108
seconds += c.base.config.PollTime
111109
if seconds > c.base.config.Timeout {
112-
return fmt.Errorf("[Error] Resource deletion timed out for %v [type: %v project: %v zone: %v] (%v seconds)", firewallID, c.Name(), c.base.config.Project, zone, c.base.config.Timeout)
110+
return fmt.Errorf("[Error] Resource deletion timed out for %v [type: %v project: %v] (%v seconds)", firewallID, c.Name(), c.base.config.Project, c.base.config.Timeout)
113111
}
114112
}
115113
c.resourceMap.Delete(firewallID)
116114

117-
log.Printf("[Info] Resource deleted %v [type: %v project: %v zone: %v] (%v seconds)", firewallID, c.Name(), c.base.config.Project, zone, seconds)
115+
log.Printf("[Info] Resource deleted %v [type: %v project: %v] (%v seconds)", firewallID, c.Name(), c.base.config.Project, seconds)
118116
return nil
119117
})
120118
return true

0 commit comments

Comments
 (0)