Skip to content

Commit

Permalink
Remove target group name (#17)
Browse files Browse the repository at this point in the history
The target group name was added for readability but was orignially
excluded because of problems destroying a target group while a listener
was attached. See comment in code. This was removed again in case this
was still an issue and becuase of a length limit of 32 chars for the
name parameter which was causing problems
  • Loading branch information
colincoleman authored Aug 4, 2023
1 parent 6da0b28 commit 7ba9064
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,6 @@ resource "aws_security_group_rule" "egress_service" {
# LB Target group
# ------------------------------------------------------------------------------
resource "aws_lb_target_group" "task" {
name = "${var.name_prefix}-${var.task_container_port}"
count = var.lb_arn == "" ? 0 : 1
vpc_id = var.vpc_id
protocol = var.task_container_protocol
Expand Down

0 comments on commit 7ba9064

Please sign in to comment.