feat: include log for each database name in export workflow (#407)

Co-authored-by: g-awmalik <malik.awais@gmail.com>

modules/backup/README.md

@@ -61,6 +61,7 @@ fetch workflows.googleapis.com/Workflow
 | export\_databases | The list of databases that should be exported - if is an empty set all databases will be exported | `set(string)` | `[]` | no |
 | export\_schedule | The cron schedule to execute the export to GCS | `string` | `"15 3 * * *"` | no |
 | export\_uri | The bucket and path uri for exporting to GCS | `string` | n/a | yes |
+| log\_db\_name\_to\_export | Whether or not to log database name in the export workflow | `bool` | `false` | no |
 | project\_id | The project ID | `string` | n/a | yes |
 | region | The region where to run the workflow | `string` | `"us-central1"` | no |
 | scheduler\_timezone | The Timezone in which the Scheduler Jobs are triggered | `string` | `"Etc/GMT"` | no |

modules/backup/main.tf

@@ -110,6 +110,7 @@ resource "google_workflows_workflow" "sql_export" {
     gcsBucket           = var.export_uri
     dbType              = split("_", data.google_sql_database_instance.backup_instance.database_version)[0]
     compressExport      = var.compress_export
+    logDbName           = var.log_db_name_to_export
   })
 }
 

modules/backup/templates/export.yaml.tftpl

@@ -43,6 +43,12 @@ main:
         value: database
         in: $${databases}
         steps:
+        %{ if logDbName }
+          - log:
+              call: sys.log
+              args:
+                data: $${"Starting backup from database " + database}
+        %{ endif }
           - export database:
               call: googleapis.sqladmin.v1.instances.export
               args:
@@ -60,6 +66,12 @@ main:
         value: database
         in: $${databases}
         steps:
+        %{ if logDbName }
+          - log:
+              call: sys.log
+              args:
+                data: $${"Starting backup from database " + database}
+        %{ endif }
           - exclude System DB:
               switch:
                 - condition: $${database != "tempdb" } #tempdb has to be excluded in an export

modules/backup/variables.tf

@@ -102,3 +102,9 @@ variable "unique_suffix" {
   type        = string
   default     = ""
 }
+
+variable "log_db_name_to_export" {
+  description = "Whether or not to log database name in the export workflow"
+  type        = bool
+  default     = false
+}