Merge pull request #17 from QuiNovas/develop

Version update and resource update

s3-bucket.tf

@@ -1,30 +1,41 @@
 resource "aws_s3_bucket" "config" {
-  acl    = "log-delivery-write"
   bucket = local.config_name
 
   lifecycle {
     prevent_destroy = true
   }
+
+  tags = var.tags
+}
 
-  lifecycle_rule {
-    id      = "log"
-    enabled = true
+resource "aws_s3_bucket_logging" "config" {
+  bucket = aws_s3_bucket.config.id
 
-    transition {
-      days          = var.transition_to_glacier
-      storage_class = "GLACIER"
-    }
+  target_bucket = var.log_bucket
+  target_prefix = "s3/${local.config_name}/"
+}
+
+resource "aws_s3_bucket_acl" "config" {
+  bucket = aws_s3_bucket.config.id
+  acl    = "log-delivery-write"
+}
+
+resource "aws_s3_bucket_lifecycle_configuration" "config" {
+  bucket = aws_s3_bucket.config.bucket
+  rule {
+    id = "log"
 
     expiration {
       days = var.expiration
     }
-  }
 
-  logging {
-    target_bucket = var.log_bucket
-    target_prefix = "s3/${local.config_name}/"
+    status = "Enabled"
+
+    transition {
+      days          = var.transition_to_glacier
+      storage_class = "GLACIER"
+    }
   }
-  tags = var.tags
 }
 
 data "aws_iam_policy_document" "config" {

versions.tf

@@ -4,6 +4,7 @@ terraform {
   required_providers {
     aws = {
       source = "hashicorp/aws"
+      version = ">= 4.0.0"
     }
   }
 }