add option to override the S3 backup bucket name

Author: briskt

modules/032-db-backup/main.tf

@@ -7,6 +7,7 @@ locals {
       "arn:aws:rds:${local.aws_region}:${local.aws_account}:db:idp-${var.idp_name}-${var.app_env}"
     )
   )
+  s3_backup_bucket = coalesce(var.s3_backup_bucket, "${var.idp_name}-${var.app_name}-${var.app_env}")
 }
 
 
@@ -22,7 +23,7 @@ data "aws_region" "current" {}
  * Create S3 bucket for storing backups
  */
 resource "aws_s3_bucket" "backup" {
-  bucket        = "${var.idp_name}-${var.app_name}-${var.app_env}"
+  bucket        = local.s3_backup_bucket
   force_destroy = true
 
   tags = {

modules/032-db-backup/variables.tf

@@ -92,6 +92,15 @@ variable "rds_arn" {
   default     = ""
 }
 
+variable "s3_backup_bucket" {
+  description = <<-EOT
+    The name of the S3 bucket to use for backup storage. If not specified, a bucket will be created with the name
+    {var.idp_name}-{var.app_name}-{var.app_env}.
+  EOT
+  type        = string
+  default     = ""
+}
+
 variable "service_mode" {
   description = "Service mode, either `backup` or `restore`"
   type        = string