Skip to content

Commit

Permalink
[Core][Deployment][ECS] same name for 2 iam roles (#854)
Browse files Browse the repository at this point in the history
  • Loading branch information
erz4 authored Jul 31, 2024
1 parent 3af86b4 commit e45ffda
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions deployment/terraform/aws/ecs/modules/ecs_service/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ data "aws_iam_policy_document" "task_execution_role_policy" {
}

resource "aws_iam_role" "task_role" {
name = local.service_name
name = "ecs-${local.service_name}"
assume_role_policy = data.aws_iam_policy_document.ecs_assume_role_policy.json
}

Expand All @@ -127,7 +127,7 @@ resource "aws_iam_policy" "execution-policy" {
}

resource "aws_iam_role" "task_execution_role" {
name = local.service_name
name = "ecs-execution-${local.service_name}"
assume_role_policy = data.aws_iam_policy_document.ecs_assume_role_policy.json
}

Expand Down

0 comments on commit e45ffda

Please sign in to comment.