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 (#3495)
  • Loading branch information
Bryant Biggs authored and GitHub committed Aug 26, 2025
1 parent 23a67d0 commit fa1d422
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/self-managed-node-group/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -433,7 +433,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 fa1d422

Please sign in to comment.