Skip to content

Commit 0c7fec0

Browse files
fix: Align EMR EKS Job Execution role with AWS docs (#38)
Co-authored-by: Bryant Biggs <[email protected]>
1 parent cd85386 commit 0c7fec0

File tree

3 files changed

+1
-14
lines changed

3 files changed

+1
-14
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.98.0
3+
rev: v1.99.1
44
hooks:
55
- id: terraform_fmt
66
- id: terraform_docs

modules/virtual-cluster/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ No modules.
112112
| [aws_caller_identity.current](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/caller_identity) | data source |
113113
| [aws_iam_policy_document.assume](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_policy_document) | data source |
114114
| [aws_iam_policy_document.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_policy_document) | data source |
115-
| [aws_partition.current](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/partition) | data source |
116115

117116
## Inputs
118117

modules/virtual-cluster/main.tf

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
data "aws_caller_identity" "current" {}
2-
data "aws_partition" "current" {}
32

43
locals {
54
account_id = data.aws_caller_identity.current.account_id
@@ -146,17 +145,6 @@ locals {
146145
data "aws_iam_policy_document" "assume" {
147146
count = local.create_iam_role ? 1 : 0
148147

149-
statement {
150-
sid = "EMR"
151-
effect = "Allow"
152-
actions = ["sts:AssumeRole"]
153-
154-
principals {
155-
type = "Service"
156-
identifiers = ["elasticmapreduce.${data.aws_partition.current.dns_suffix}"]
157-
}
158-
}
159-
160148
statement {
161149
sid = "IRSA"
162150
effect = "Allow"

0 commit comments

Comments
 (0)