From 99f48dd41ae271b611fa774b8b135e82b3ed5959 Mon Sep 17 00:00:00 2001
From: pehan <pehan@dfds.com>
Date: Thu, 9 Jan 2025 09:29:11 +0100
Subject: [PATCH] fix: Remove conflicting S3 bucket ACL resource from Terraform
 configuration
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

If “Bucket owner enforced” is active for your S3 bucket, ACLs are effectively disabled. Attempting to set them via Terraform (aws_s3_bucket_acl) will fail with AccessControlListNotSupported. See https://github.com/hashicorp/terraform-provider-aws/issues/34079
---
 mlflow-terraform/main.tf | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/mlflow-terraform/main.tf b/mlflow-terraform/main.tf
index f62b5bb..1c7c4dd 100644
--- a/mlflow-terraform/main.tf
+++ b/mlflow-terraform/main.tf
@@ -3,11 +3,7 @@
 provider "aws" {
   region = "eu-central-1"
 }
-// Mlflow bucket will get a random unique name
-resource "aws_s3_bucket_acl" "mlflow_bucket_acl" {
-  acl = "private"
-  bucket = aws_s3_bucket.mlflow_bucket.id
-}
+
 resource "aws_s3_bucket" "mlflow_bucket" {
   tags = {
     Name = "Mlflow model artifacts bucket"