Skip to content

Commit f0d00a9

Browse files
committed
Revert "Use KMS where there's no SSE fallback"
This reverts commit 4460b9c. If you want this encrypted, you can just turn on kms. This keeps parity with Ashok's customer, and makes the options more literally honest.
1 parent 4460b9c commit f0d00a9

File tree

1 file changed

+4
-4
lines changed
  • modules/infra/submodules/cost-usage-report

1 file changed

+4
-4
lines changed

modules/infra/submodules/cost-usage-report/glue.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,13 @@ resource "aws_glue_security_configuration" "lambda_config" {
1515

1616
encryption_configuration {
1717
cloudwatch_encryption {
18-
cloudwatch_encryption_mode = "SSE-KMS"
19-
kms_key_arn = var.kms_info.key_arn
18+
cloudwatch_encryption_mode = var.kms_info.enabled ? "SSE-KMS" : "DISABLED"
19+
kms_key_arn = local.kms_key_arn
2020
}
2121

2222
job_bookmarks_encryption {
23-
job_bookmarks_encryption_mode = "CSE-KMS"
24-
kms_key_arn = var.kms_info.key_arn
23+
job_bookmarks_encryption_mode = var.kms_info.enabled ? "CSE-KMS" : "DISABLED"
24+
kms_key_arn = local.kms_key_arn
2525
}
2626

2727
s3_encryption {

0 commit comments

Comments
 (0)