-
-
Notifications
You must be signed in to change notification settings - Fork 156
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ALB register container instances supported? #39
Comments
yes it does. |
The The https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/blob/master/main.tf#L308 But I don't see a |
I came across the same question, the only answer was pretty much this ticket |
@BGarber42 @archenroot Oh I was mistaken. You do not need a target group attachment. I believe only the listener rule (created by the |
I just ran into this again. I'm glad this ticket was written because the above description reminded me that I do not need a specific If you want to create an attachment outside of this module, the following can be done. module "ecs_web_app" {
# ...
}
resource "aws_lb_target_group_attachment" "default" {
target_group_arn = module.ecs_web_app.alb_ingress_target_group_arn
# Replace with your target_id
# target_id = null
} There are very few inputs to this and we could add this as an optional resource within this module but it's unnecessary if you rely on listener rules to direct directly to the LB. The attachment only seems necessary if the listener rule directs directly to the target group resource "aws_lb_listener" "default" {
default_action {
type = "forward"
target_group_arn = module.ecs_web_app.alb_ingress_target_group_arn
}
} |
Hi,
I don't see any aws_lb_target_group_attachment resource being used, but it can be hidden somewhere in submodules being used.
Does this module cover registering container instances in ALB target group?
The text was updated successfully, but these errors were encountered: