Add point in time recovery option in backend coniguration

Author: mvoitko

modules/cloud-sql/main.tf

@@ -64,9 +64,10 @@ resource "google_sql_database_instance" "master" {
     }
 
     backup_configuration {
-      binary_log_enabled = local.actual_binary_log_enabled
-      enabled            = var.backup_enabled
-      start_time         = var.backup_start_time
+      binary_log_enabled             = local.actual_binary_log_enabled
+      enabled                        = var.backup_enabled
+      start_time                     = var.backup_start_time
+      point_in_time_recovery_enabled = local.is_postgres ? null : var.postgres_point_in_time_recovery_enabled
     }
 
     maintenance_window {
@@ -121,7 +122,7 @@ resource "google_sql_user" "default" {
   name     = var.master_user_name
   instance = google_sql_database_instance.master.name
   # Postgres users don't have hosts, so the API will ignore this value which causes Terraform to attempt
-  # to recreate the user each time.
+  # to recreate the user each time
   # See https://github.com/terraform-providers/terraform-provider-google/issues/1526 for more information.
   host     = local.is_postgres ? null : var.master_user_host
   password = var.master_user_password

modules/cloud-sql/variables.tf

@@ -87,6 +87,12 @@ variable "backup_start_time" {
   default     = "04:00"
 }
 
+variable "postgres_point_in_time_recovery_enabled" {
+  description = "Will restart database if enabled after instance creation - only applicable to PostgreSQL"
+  type        = bool
+  default     = false
+}
+
 variable "mysql_binary_log_enabled" {
   description = "Set to false if you want to disable binary logs - only applicable to MySQL. Note, when using failover or read replicas, master and existing backups need to have binary_log_enabled=true set."
   type        = bool