remove s3 acl from terraform/addons/logging-alb (#12169)

Fixes "the bucket does not allow acls". If it doesn't allow them, then
don't make one.

Validated logging still works.
This commit is contained in:
Robert Fairburn
2023-06-06 14:42:04 -04:00
committed by GitHub
parent f27fcddd55
commit cfe59b169e
+2 -4
View File
@@ -39,10 +39,9 @@ resource "aws_kms_alias" "logs_alias" {
module "s3_bucket_for_logs" {
source = "terraform-aws-modules/s3-bucket/aws"
version = "3.6.0"
version = "3.11.0"
bucket = "${var.prefix}-alb-logs"
acl = "log-delivery-write"
# Allow deletion of non-empty bucket
force_destroy = true
@@ -92,10 +91,9 @@ resource "aws_athena_database" "logs" {
module "athena-s3-bucket" {
count = var.enable_athena == true ? 1 : 0
source = "terraform-aws-modules/s3-bucket/aws"
version = "3.6.0"
version = "3.11.0"
bucket = "${var.prefix}-alb-logs-athena"
acl = "log-delivery-write"
# Allow deletion of non-empty bucket
force_destroy = true