Skip to content

Commit b79c17a

Browse files
authored
fix: Do not create dynamic DNS suffix for service principals (terraform-aws-modules#442)
1 parent b4b48fd commit b79c17a

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
repos:
22
- repo: https://github.com/antonbabenko/pre-commit-terraform
3-
rev: v1.88.0
3+
rev: v1.88.4
44
hooks:
55
- id: terraform_fmt
66
- id: terraform_docs

main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ data "aws_iam_policy_document" "monitoring_rds_assume_role" {
238238

239239
principals {
240240
type = "Service"
241-
identifiers = ["monitoring.rds.${data.aws_partition.current.dns_suffix}"]
241+
identifiers = ["monitoring.rds.amazonaws.com"]
242242
}
243243
}
244244
}

0 commit comments

Comments
 (0)