Fix short_term_retention_days as optional

aztfmod · Sep 18, 2023 · 3101f44 · 3101f44 · SimonSouli · Sep 19, 2023
1 parent 9040a9c
commit 3101f44
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/modules/databases/mssql_managed_database_v1/var/settings/srtdays.tf b/modules/databases/mssql_managed_database_v1/var/settings/srtdays.tf
@@ -1,9 +1,9 @@
 variable "short_term_retention_days" {
-  description = "short_term_retention_days .The backup retention period in days. This is how many days Point-in-Time Restore will be supported. Value must be atleast 1"
-  nullable    = false
+  description = "short_term_retention_days (Optional) .The backup retention period in days. This is how many days Point-in-Time Restore will be supported. Value must be atleast 1"
+  nullable    = true
 
   validation {
-    condition = var.short_term_retention_days >= 1
+    condition = var.short_term_retention_day == null ? true : var.short_term_retention_days >= 1
 
     error_message = format("Not supported value: '%s'. \nAdjust your configuration file with a supported value of 1 or more days", var.short_term_retention_days)
   }