From 4494e1cec95c86f6f541bcbb890749209f458916 Mon Sep 17 00:00:00 2001 From: Steven Martin Date: Mon, 8 May 2023 16:19:22 -0400 Subject: [PATCH] enable acl in single aws terraform s3 (#25847) * update single aws terraform s3 * terraform lint fix --- examples/aws/terraform/starter-cluster/s3.tf | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/examples/aws/terraform/starter-cluster/s3.tf b/examples/aws/terraform/starter-cluster/s3.tf index 4757fa1b960..7d3b492d8e6 100644 --- a/examples/aws/terraform/starter-cluster/s3.tf +++ b/examples/aws/terraform/starter-cluster/s3.tf @@ -13,8 +13,17 @@ resource "aws_s3_bucket" "storage" { } resource "aws_s3_bucket_acl" "storage" { - bucket = aws_s3_bucket.storage.bucket - acl = "private" + depends_on = [aws_s3_bucket_ownership_controls.storage] + bucket = aws_s3_bucket.storage.bucket + acl = "private" +} + +resource "aws_s3_bucket_ownership_controls" "storage" { + bucket = aws_s3_bucket.storage.id + + rule { + object_ownership = "BucketOwnerPreferred" + } } // For demo purposes, CMK is not needed