Skip to content

Commit 50ed343

Browse files
fix max-length for nodegroup IAM role
1 parent 835d3f6 commit 50ed343

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

pkg/cloud/services/eks/roles.go

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,12 +156,20 @@ func (s *NodegroupService) reconcileNodegroupIAMRole() error {
156156

157157
if s.scope.RoleName() == "" {
158158
var roleName string
159+
var err error
159160
if !s.scope.EnableIAM() {
160161
s.scope.Info("no EKS nodegroup role specified, using default EKS nodegroup role")
161162
roleName = infrav1exp.DefaultEKSNodegroupRole
162163
} else {
163164
s.scope.Info("no EKS nodegroup role specified, using role based on nodegroup name")
164-
roleName = fmt.Sprintf("%s-%s-nodegroup-iam-service-role", s.scope.KubernetesClusterName(), s.scope.NodegroupName())
165+
roleName, err = eks.GenerateEKSName(
166+
fmt.Sprintf("%s-%s", s.scope.KubernetesClusterName(), s.scope.NodegroupName()),
167+
"-nodegroup-iam-service-role",
168+
maxIAMRoleNameLength,
169+
)
170+
if err != nil {
171+
return errors.Wrap(err, "failed to generate IAM role name")
172+
}
165173
}
166174
s.scope.ManagedMachinePool.Spec.RoleName = roleName
167175
}

0 commit comments

Comments
 (0)