Skip to content

Commit 934c478

Browse files
committed
Fix resource names and use arn instead of ID
1 parent dcb1f24 commit 934c478

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ performance and health.
7373
- resource.aws_cloudwatch_log_group.main (main.tf#73)
7474
- resource.aws_ecs_cluster.main (main.tf#10)
7575
- data source.aws_caller_identity.current (main.tf#100)
76-
- data source.aws_iam_policy_document.kms_ephemeral (main.tf#101)
76+
- data source.aws_iam_policy_document.kms_storage (main.tf#101)
7777

7878
# Examples
7979
### Basic Example

main.tf

+5-5
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ resource "aws_ecs_cluster" "main" {
5050
for_each = var.encrypt_ephemeral_storage || var.encrypt_managed_storage ? [true] : []
5151

5252
content {
53-
kms_key_id = var.encrypt_managed_storage ? module.kms[0].key_id : null
54-
fargate_ephemeral_storage_kms_key_id = var.encrypt_ephemeral_storage ? module.kms_ephemeral[0].key_id : null
53+
kms_key_id = var.encrypt_managed_storage ? module.kms_storage[0].key_arn : null
54+
fargate_ephemeral_storage_kms_key_id = var.encrypt_ephemeral_storage ? module.kms_storage[0].key_arn : null
5555
}
5656
}
5757
}
@@ -86,19 +86,19 @@ resource "aws_cloudwatch_log_group" "container_insights" {
8686
tags = var.tags
8787
}
8888

89-
module "kms_ephemeral" {
89+
module "kms_storage" {
9090
count = var.encrypt_ephemeral_storage || var.encrypt_managed_storage ? 1 : 0
9191

9292
source = "geekcell/kms/aws"
9393
version = ">= 1.0.0, < 2.0.0"
94-
policy = data.aws_iam_policy_document.kms_ephemeral[0].json
94+
policy = data.aws_iam_policy_document.kms_storage[0].json
9595

9696
alias = "ecs/cluster/${var.name}/managed-storage"
9797
tags = var.tags
9898
}
9999

100100
data "aws_caller_identity" "current" {}
101-
data "aws_iam_policy_document" "kms_ephemeral" {
101+
data "aws_iam_policy_document" "kms_storage" {
102102
count = var.encrypt_ephemeral_storage || var.encrypt_managed_storage ? 1 : 0
103103

104104
statement {

0 commit comments

Comments
 (0)