Fix s3 bucket AccessControlListNotSupported error (#72)

* Fix s3 bucket AccessControlListNotSupported error

* Apply depends_on to s3 bucket to prevent AccessControlListNotSupported

cloud_AWS/terraform/module/s3.tf

@@ -8,6 +8,8 @@ resource "aws_s3_bucket_acl" "acl" {
   count  = (var.s3_use_one_bucket == false ? length(var.vpc_id_list) : 1)
   bucket = aws_s3_bucket.vpc_logs[count.index].id
   acl    = "private"
+  # This `depends_on` is to prevent "AccessControlListNotSupported: The bucket does not allow ACLs."
+  depends_on = [aws_s3_bucket_ownership_controls.ownership]
 }
 
 resource "aws_s3_bucket_policy" "policy" {
@@ -19,6 +21,14 @@ resource "aws_s3_bucket_policy" "policy" {
   })
 }
 
+resource "aws_s3_bucket_ownership_controls" "ownership" {
+  count  = (var.s3_use_one_bucket == false ? length(var.vpc_id_list) : 1)
+  bucket = aws_s3_bucket.vpc_logs[count.index].id
+  rule {
+    object_ownership = "ObjectWriter"
+  }
+}
+
 resource "aws_s3_bucket_public_access_block" "vpc_logs" {
   count                   = (var.s3_use_one_bucket == false ? length(var.vpc_id_list) : 1)
   bucket                  = aws_s3_bucket.vpc_logs[count.index].id