Skip to content

Commit

Permalink
fix: Ensure module created security group is included on any network …
Browse files Browse the repository at this point in the history
…interfaces created (#3493)

* fix(module): wrong variable name in eks-managed-node-group

* Update modules/eks-managed-node-group/main.tf

---------

Co-authored-by: Bryant Biggs <bryantbiggs@gmail.com>
  • Loading branch information
2 people authored and GitHub committed Aug 25, 2025
1 parent e5c35cf commit e5cff84
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/eks-managed-node-group/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ resource "aws_launch_template" "this" {
primary_ipv6 = network_interfaces.value.primary_ipv6
private_ip_address = network_interfaces.value.private_ip_address
# Ref: https://github.com/hashicorp/terraform-provider-aws/issues/4570
security_groups = compact(concat(network_interfaces.value.security_groups, var.vpc_security_group_ids))
security_groups = compact(concat(network_interfaces.value.security_groups, local.security_group_ids))
# Set on EKS managed node group, will fail if set here
# https://docs.aws.amazon.com/eks/latest/userguide/launch-templates.html#launch-template-basics
# subnet_id = try(network_interfaces.value.subnet_id, null)
Expand Down

0 comments on commit e5cff84

Please sign in to comment.