github cloudposse/terraform-aws-eks-node-group 0.24.0
v0.24.0 Unstable Pre-Release

latest releases: v3.1.1, v3, v3.1.0...
pre-release3 years ago

See note in Release v0.21.0 (https://github.com/cloudposse/terraform-aws-eks-node-group/releases/tag/0.21.0)

Always add var.security_groups to launch template if provided @cvittoriasona (#77)

what

  • If var.security_groups is present, add any passed in security groups, along with the default cluster security group, to the launch template.

why

  • var.security_groups is only added to the launch template if var.remote_access_enabled is true. Additional security groups should not be dependent on SSH access being enabled to be used.
  • Specifically, ran into an issue when using a x-account shared VPC where the default security group for the VPC was not available to accounts the VPC was shared with. After encountering this error, when attempting to specify a security group for the launch template using var.security_groups, realized this var isn't active unless var.remote_access_enabled is also set. See below for output:
Error: error creating EKS Node Group (my-eks-node-group): InvalidRequestException: You do not have access to a default security group in VPC vpc-123456. Specify a security group, 310. Specify a security group, and try again.
│ {
│   RespMetadata: {
│     StatusCode: 400,
│     RequestID: "some-uuid"
│   },
│   Message_: "You do not have access to a default security group in VPC vpc-123456. Specify a security group, and try again."
│ }

This seems to be mostly a workaround for launch templates as EKS managed nodegroups should be auto-assigned to the default cluster security group, even if the launch template has no security groups attached to it.

Issue was present in v0.19.0 only when using var.kubernetes_taints, but in >=v0.20.0 this issue applied to all nodegroups created with this module.

references

  • Tested with AWS provider v3.44.0 & v3.50.0

Don't miss a new terraform-aws-eks-node-group release

NewReleases is sending notifications on new releases.