Skip to content

Commit

Permalink
fix: add missing tag variable in autoscaling target resource
Browse files Browse the repository at this point in the history
Signed-off-by: Lays Rodrigues <[email protected]>
  • Loading branch information
lays147 committed Nov 16, 2023
1 parent 95546b5 commit a788de1
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion modules/service/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -522,7 +522,7 @@ resource "aws_iam_role_policy_attachment" "service" {
module "container_definition" {
source = "../container-definition"

for_each = { for k, v in var.container_definitions : k => v if local.create_task_definition && try(v.create, true)}
for_each = { for k, v in var.container_definitions : k => v if local.create_task_definition && try(v.create, true) }

operating_system_family = try(var.runtime_platform.operating_system_family, "LINUX")

Expand Down Expand Up @@ -1202,6 +1202,7 @@ resource "aws_appautoscaling_target" "this" {
resource_id = "service/${local.cluster_name}/${try(aws_ecs_service.this[0].name, aws_ecs_service.ignore_task_definition[0].name)}"
scalable_dimension = "ecs:service:DesiredCount"
service_namespace = "ecs"
tags = var.tags
}

resource "aws_appautoscaling_policy" "this" {
Expand Down

0 comments on commit a788de1

Please sign in to comment.