[GH-ISSUE #111] Target Group returned instead of load balancer #57

New issue

Closed

opened 2026-05-06 12:36:48 +02:00 by BreizhHardware · 1 comment

BreizhHardware commented

2026-05-06 12:36:48 +02:00

Owner

Originally created by @ludevrie on GitHub (Mar 13, 2024).
Original GitHub issue: https://github.com/patrickchugh/terravision/issues/111

This happens in an ECS Fargate setup with internal and external ALB load balancers.

In graphmaker handle_special_resources() crashes when looking for "aws_lb".
The reason is that in resource_handlers.aws_handle_lb() the variable in found_lbs (['aws_lb.elb']) contains the details of a target group instead of a load balancer.
This in turn causes helpers.check_variant(lb, tfdata["meta_data"][lb]) to fail, because the meta_data contains neither "application" nor "network" and False is returned (which causes the crash, because on the next line it is concatenated with a string (renamed_node = lb_type + "." + "elb")

Originally created by @ludevrie on GitHub (Mar 13, 2024). Original GitHub issue: https://github.com/patrickchugh/terravision/issues/111 This happens in an ECS Fargate setup with internal and external ALB load balancers. In graphmaker handle_special_resources() crashes when looking for "aws_lb". The reason is that in resource_handlers.aws_handle_lb() the variable in found_lbs (['aws_lb.elb']) contains the details of a target group instead of a load balancer. This in turn causes helpers.check_variant(lb, tfdata["meta_data"][lb]) to fail, because the meta_data contains neither "application" nor "network" and False is returned (which causes the crash, because on the next line it is concatenated with a string (`renamed_node = lb_type + "." + "elb"`)

BreizhHardware closed this issue

2026-05-06 12:36:49 +02:00

BreizhHardware commented

2026-05-06 12:36:50 +02:00

Author

Owner

@patrickchugh commented on GitHub (Mar 22, 2026):

@ludevrie There have been many changes to the code since this issue was opened. Closing for now and please open a new issue if this still occurs.

@patrickchugh commented on GitHub (Mar 22, 2026): @ludevrie There have been many changes to the code since this issue was opened. Closing for now and please open a new issue if this still occurs.

BreizhHardware referenced this issue

2026-05-06 12:37:52 +02:00

[PR #57] [CLOSED] Add Dockerfile #149