Skip to content

Commit

Permalink
Name tag should be first in merge
Browse files Browse the repository at this point in the history
  • Loading branch information
dex4er committed Sep 9, 2022
1 parent 5d20ea1 commit 6076d19
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions iam.tf
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ resource "aws_iam_role" "eventbridge" {
permissions_boundary = var.role_permissions_boundary
assume_role_policy = data.aws_iam_policy_document.assume_role[0].json

tags = merge(var.tags, { Name = local.role_name }, var.role_tags)
tags = merge({ Name = local.role_name }, var.tags, var.role_tags)
}

#####################
Expand Down Expand Up @@ -350,7 +350,7 @@ resource "aws_iam_policy" "additional_json" {
path = var.role_path
policy = var.policy_json

tags = merge(var.tags, { Name = local.role_name })
tags = merge({ Name = local.role_name }, var.tags)
}

resource "aws_iam_policy_attachment" "additional_json" {
Expand Down

0 comments on commit 6076d19

Please sign in to comment.