Made all lifecycle_rules optional

zachreborn · Mar 28, 2024 · 4c556a2 · 4c556a2
1 parent bc19376
commit 4c556a2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/modules/aws/s3/bucket/variables.tf b/modules/aws/s3/bucket/variables.tf
@@ -178,7 +178,7 @@ variable "intelligent_tiering_days" {
 ######################
 
 variable "lifecycle_rules" {
-  description = "(Optional) Configuration of object lifecycle management (LCM). Can have several rules as a list of maps where each map is the lifecycle rule configuration. Type should be list(map(string))."
+  description = "(Optional) Configuration of object lifecycle management (LCM). Can have several rules as a list of maps where each map is the lifecycle rule configuration."
   type = list(object({
     id     = string
     status = string
@@ -200,11 +200,11 @@ variable "lifecycle_rules" {
       newer_noncurrent_versions = optional(number)
       noncurrent_days           = optional(number)
     }))
-    noncurrent_version_transitions = list(object({
+    noncurrent_version_transitions = optional(list(object({
       newer_noncurrent_versions = optional(number)
       noncurrent_days           = optional(number)
       storage_class             = optional(string)
-    }))
+    })))
     transition = list(object({
       date          = optional(string)
       days          = optional(number)

diff --git a/modules/aws/transfer_family/variables.tf b/modules/aws/transfer_family/variables.tf
@@ -177,7 +177,7 @@ variable "vpc_id" {
 ###########################
 
 variable "lifecycle_rules" {
-  description = "(Optional) A list of lifecycle rules to apply to the bucket"
+  description = "(Optional) Configuration of object lifecycle management (LCM). Can have several rules as a list of maps where each map is the lifecycle rule configuration."
   type = list(object({
     id     = string
     status = string
@@ -199,11 +199,11 @@ variable "lifecycle_rules" {
       newer_noncurrent_versions = optional(number)
       noncurrent_days           = optional(number)
     }))
-    noncurrent_version_transitions = list(object({
+    noncurrent_version_transitions = optional(list(object({
       newer_noncurrent_versions = optional(number)
       noncurrent_days           = optional(number)
       storage_class             = optional(string)
-    }))
+    })))
     transition = list(object({
       date          = optional(string)
       days          = optional(number)