feat(postgres): Added point_in_time_recovery_enabled backup option fo…

…r Postgres (#142)
terraform-google-modules · Oct 23, 2020 · 1fcae8a · 1fcae8a
1 parent 69e1911
commit 1fcae8a
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 13 deletions.
diff --git a/examples/postgresql-ha/main.tf b/examples/postgresql-ha/main.tf
@@ -75,9 +75,10 @@ module "pg" {
   }
 
   backup_configuration = {
-    enabled    = true
-    start_time = "20:55"
-    location   = null
+    enabled                        = true
+    start_time                     = "20:55"
+    location                       = null
+    point_in_time_recovery_enabled = false
   }
 
   // Read replica configurations

diff --git a/modules/postgresql/main.tf b/modules/postgresql/main.tf
@@ -51,10 +51,11 @@ resource "google_sql_database_instance" "default" {
     dynamic "backup_configuration" {
       for_each = [var.backup_configuration]
       content {
-        binary_log_enabled = false
-        enabled            = lookup(backup_configuration.value, "enabled", null)
-        start_time         = lookup(backup_configuration.value, "start_time", null)
-        location           = lookup(backup_configuration.value, "location", null)
+        binary_log_enabled             = false
+        enabled                        = lookup(backup_configuration.value, "enabled", null)
+        start_time                     = lookup(backup_configuration.value, "start_time", null)
+        location                       = lookup(backup_configuration.value, "location", null)
+        point_in_time_recovery_enabled = lookup(backup_configuration.value, "point_in_time_recovery_enabled", false)
       }
     }
     dynamic "ip_configuration" {

diff --git a/modules/postgresql/variables.tf b/modules/postgresql/variables.tf
@@ -125,14 +125,16 @@ variable "user_labels" {
 variable "backup_configuration" {
   description = "The backup_configuration settings subblock for the database setings"
   type = object({
-    enabled    = bool
-    start_time = string
-    location   = string
+    enabled                        = bool
+    start_time                     = string
+    location                       = string
+    point_in_time_recovery_enabled = bool
   })
   default = {
-    enabled    = false
-    start_time = null
-    location   = null
+    enabled                        = false
+    start_time                     = null
+    location                       = null
+    point_in_time_recovery_enabled = false
   }
 }