Skip to content

Commit c659a22

Browse files
committed
OCPBUGS-34713: AWS remove extra tags from awsmachines for security groups and subnets
** Manifests were generated to work for both CAPI and Terraform installs. Now that CAPI is the base, the extra tags for security groups and subnets can be removed from awsmachines.
1 parent 1067870 commit c659a22

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

pkg/asset/machines/aws/machines.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,6 @@ func Machines(clusterID string, region string, subnets map[string]string, pool *
115115
{
116116
Name: "tag:Name",
117117
Values: []string{
118-
fmt.Sprintf("%s-private-%s", clusterID, zone), // legacy Terraform config, TODO remove
119118
fmt.Sprintf("%s-subnet-private-%s", clusterID, zone),
120119
},
121120
},
@@ -184,10 +183,6 @@ func provider(in *machineProviderInput) (*machineapi.AWSMachineProviderConfig, e
184183
}
185184

186185
sgFilters := []machineapi.Filter{
187-
{
188-
Name: "tag:Name",
189-
Values: []string{fmt.Sprintf("%s-%s-sg", in.clusterID, in.role)}, // legacy Terraform config, remove with Terraform
190-
},
191186
{
192187
Name: "tag:Name",
193188
Values: []string{fmt.Sprintf("%s-node", in.clusterID)},
@@ -261,7 +256,6 @@ func provider(in *machineProviderInput) (*machineapi.AWSMachineProviderConfig, e
261256
{
262257
Name: "tag:Name",
263258
Values: []string{
264-
fmt.Sprintf("%s-%s-%s", in.clusterID, visibility, in.zone), // legacy Terraform config, remove with Terraform
265259
fmt.Sprintf("%s-subnet-%s-%s", in.clusterID, visibility, in.zone),
266260
},
267261
},

0 commit comments

Comments
 (0)