Skip to content

Commit

Permalink
fixing issue where the same security group name is global running and…
Browse files Browse the repository at this point in the history
… staging (#538)
  • Loading branch information
Caleb Washburn committed Apr 23, 2024
1 parent e050d4b commit 28a895a
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions securitygroup/securitygroup.go
Original file line number Diff line number Diff line change
Expand Up @@ -501,6 +501,7 @@ func (m *DefaultManager) AssignSecurityGroupGlobalRunning(sg *resource.SecurityG
r := &resource.SecurityGroupUpdate{
GloballyEnabled: &resource.SecurityGroupGloballyEnabled{
Running: true,
Staging: sg.GloballyEnabled.Staging,
},
}
sg.GloballyEnabled.Running = true
Expand All @@ -513,10 +514,12 @@ func (m *DefaultManager) AssignSecurityGroupGlobalStaging(sg *resource.SecurityG
lo.G.Infof("[dry-run]: assigning sg %s as staging security group", sg.Name)
return nil
}

lo.G.Infof("assigning sg %s as staging security group", sg.Name)
r := &resource.SecurityGroupUpdate{
GloballyEnabled: &resource.SecurityGroupGloballyEnabled{
Staging: true,
Running: sg.GloballyEnabled.Running,
},
}
sg.GloballyEnabled.Staging = true
Expand Down

0 comments on commit 28a895a

Please sign in to comment.