🐛: revert to using hardcoded prefix for target groups

Using the LB's name as prefix has the potential of exceeding the 32
characters limit.

Author: r4f4

pkg/cloud/services/elb/loadbalancer.go

@@ -239,18 +239,6 @@ func (s *Service) getAdditionalTargetGroupHealthCheck(ln infrav1.AdditionalListe
 	return healthCheck
 }
 
-// getTargetGroupName creates the target group name based on LB Name, when defined, otherwise return
-// the standard name created from the timestamp.
-func (s *Service) getTargetGroupName(lbSpec *infrav1.AWSLoadBalancerSpec, defaultPrefix string, port int64) string {
-	targetName := fmt.Sprintf("%s-%d", defaultPrefix, time.Now().Unix())
-
-	if lbSpec != nil && lbSpec.Name != nil {
-		targetName = fmt.Sprintf("%s-%d", *lbSpec.Name, port)
-	}
-
-	return targetName
-}
-
 func (s *Service) getAPIServerLBSpec(elbName string, lbSpec *infrav1.AWSLoadBalancerSpec) (*infrav1.LoadBalancer, error) {
 	var securityGroupIDs []string
 	if lbSpec != nil {
@@ -266,7 +254,6 @@ func (s *Service) getAPIServerLBSpec(elbName string, lbSpec *infrav1.AWSLoadBala
 
 	// The default API health check is TCP, allowing customization to HTTP or HTTPS when HealthCheckProtocol is set.
 	apiHealthCheck := s.getAPITargetGroupHealthCheck(lbSpec)
-	apiTargetGroupName := s.getTargetGroupName(lbSpec, "apiserver-target", infrav1.DefaultAPIServerPort)
 	res := &infrav1.LoadBalancer{
 		Name:          elbName,
 		Scheme:        scheme,
@@ -276,7 +263,7 @@ func (s *Service) getAPIServerLBSpec(elbName string, lbSpec *infrav1.AWSLoadBala
 				Protocol: infrav1.ELBProtocolTCP,
 				Port:     infrav1.DefaultAPIServerPort,
 				TargetGroup: infrav1.TargetGroupSpec{
-					Name:        apiTargetGroupName,
+					Name:        fmt.Sprintf("apiserver-target-%d", time.Now().Unix()),
 					Port:        infrav1.DefaultAPIServerPort,
 					Protocol:    infrav1.ELBProtocolTCP,
 					VpcID:       s.scope.VPC().ID,
@@ -289,7 +276,6 @@ func (s *Service) getAPIServerLBSpec(elbName string, lbSpec *infrav1.AWSLoadBala
 
 	if lbSpec != nil {
 		for _, listener := range lbSpec.AdditionalListeners {
-			targetGroupName := s.getTargetGroupName(lbSpec, "additional-listener", listener.Port)
 			lnHealthCheck := &infrav1.TargetGroupHealthCheck{
 				Protocol: aws.String(string(listener.Protocol)),
 				Port:     aws.String(strconv.FormatInt(listener.Port, 10)),
@@ -302,7 +288,7 @@ func (s *Service) getAPIServerLBSpec(elbName string, lbSpec *infrav1.AWSLoadBala
 				Protocol: listener.Protocol,
 				Port:     listener.Port,
 				TargetGroup: infrav1.TargetGroupSpec{
-					Name:        targetGroupName,
+					Name:        fmt.Sprintf("additional-listener-%d", time.Now().Unix()),
 					Port:        listener.Port,
 					Protocol:    listener.Protocol,
 					VpcID:       s.scope.VPC().ID,

pkg/cloud/services/elb/loadbalancer_test.go

@@ -3341,63 +3341,6 @@ func stubGetBaseService(t *testing.T, clusterName string) *Service {
 	}
 }
 
-func TestService_getTargetGroupName(t *testing.T) {
-	type argWant struct {
-		value      string
-		prefixOnly bool
-	}
-	type args struct {
-		lbSpec        *infrav1.AWSLoadBalancerSpec
-		defaultPrefix string
-		port          int64
-	}
-	tests := []struct {
-		name string
-		args args
-		want argWant
-	}{
-		{
-			name: "default name",
-			args: args{
-				lbSpec:        &infrav1.AWSLoadBalancerSpec{},
-				defaultPrefix: "apiserver-target",
-				port:          6443,
-			},
-			want: argWant{
-				value:      "apiserver-target-",
-				prefixOnly: true,
-			},
-		},
-		{
-			name: "custom name",
-			args: args{
-				lbSpec: &infrav1.AWSLoadBalancerSpec{
-					Name: ptr.To("foo"),
-				},
-				defaultPrefix: "api",
-				port:          6443,
-			},
-			want: argWant{
-				value: "foo-6443",
-			},
-		},
-	}
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			s := stubGetBaseService(t, "foo")
-			if tt.want.prefixOnly {
-				if got := s.getTargetGroupName(tt.args.lbSpec, tt.args.defaultPrefix, tt.args.port); !strings.HasPrefix(got, tt.want.value) {
-					t.Errorf("Service.getTargetGroupName() = %v, wantPrefix %v", got, tt.want.value)
-				}
-				return
-			}
-			if got := s.getTargetGroupName(tt.args.lbSpec, tt.args.defaultPrefix, tt.args.port); got != tt.want.value {
-				t.Errorf("Service.getTargetGroupName() = %v, want %v", got, tt.want)
-			}
-		})
-	}
-}
-
 func TestService_getAPITargetGroupHealthCheck(t *testing.T) {
 	tests := []struct {
 		name   string