From 7932c20219e2bb7ed5d5164eb4f9a6f14a81a3b9 Mon Sep 17 00:00:00 2001
From: Desmond Ho <39398132+DesmondH0@users.noreply.github.com>
Date: Tue, 11 Jan 2022 17:17:50 +0000
Subject: [PATCH] feat: add delete_marker_replication_status in replication
 rules (#126)

Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
---
 main.tf      | 2 ++
 variables.tf | 1 +
 2 files changed, 3 insertions(+)

diff --git a/main.tf b/main.tf
index 94aa0e9c..f06e79bb 100644
--- a/main.tf
+++ b/main.tf
@@ -173,6 +173,8 @@ resource "aws_s3_bucket" "default" {
           # is required even if it empty, so we always implement `prefix` as a filter.
           # OBSOLETE: prefix   = try(rules.value.prefix, null)
           status = try(rules.value.status, null)
+          # The `Delete marker replication` was disabled by default since empty filter created in Line 210, this needed to be "Enabled" to turn it on
+          delete_marker_replication_status = try(rules.value.delete_marker_replication_status, null)
 
           destination {
             # Prefer newer system of specifying bucket in rule, but maintain backward compatibility with
diff --git a/variables.tf b/variables.tf
index 46390978..7a5c255b 100644
--- a/variables.tf
+++ b/variables.tf
@@ -190,6 +190,7 @@ variable "s3_replication_rules" {
   #   priority    = number
   #   prefix      = string
   #   status      = string
+  #   delete_marker_replication_status = string
   #   # destination_bucket is specified here rather than inside the destination object
   #   # to make it easier to work with the Terraform type system and create a list of consistent type.
   #   destination_bucket = string # destination bucket ARN, overrides s3_replica_bucket_arn