Skip to content

Commit fc770ec

Browse files
authored
feat: remove terraform resource for ELB bucket ACL configuration (terraform-only change)
Stops managing aws_s3_bucket_acl given AWS's new standard for ACLs: https://aws.amazon.com/about-aws/whats-new/2022/12/amazon-s3-automatically-enable-block-public-access-disable-access-control-lists-buckets-april-2023/ When updating, the`module.datadog.aws_s3_bucket_acl.elb_logs[0]` will be destroyed, but this will not impact the actual ACL of the bucket. Thanks, @frozenSolid!
1 parent 42b9a57 commit fc770ec

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

logs_monitoring_elb.tf

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,6 @@ resource "aws_s3_bucket_policy" "elb_logs" {
5252
policy = data.aws_iam_policy_document.elb_logs.json
5353
}
5454

55-
resource "aws_s3_bucket_acl" "elb_logs" {
56-
count = var.create_elb_logs_bucket ? 1 : 0
57-
bucket = aws_s3_bucket.elb_logs[0].id
58-
acl = "private"
59-
}
60-
6155
resource "aws_s3_bucket_lifecycle_configuration" "elb_logs" {
6256
count = var.create_elb_logs_bucket ? 1 : 0
6357
bucket = aws_s3_bucket.elb_logs[0].id

0 commit comments

Comments
 (0)