backup_retention_period (#6)

Author: jpluta

README.md

@@ -117,7 +117,7 @@ No modules.
 | <a name="input_allocated_storage"></a> [allocated\_storage](#input\_allocated\_storage) | A size of the DB storage. | `number` | `20` | no |
 | <a name="input_auto_minor_version_upgrade"></a> [auto\_minor\_version\_upgrade](#input\_auto\_minor\_version\_upgrade) | Enables minor version auto upgrade. | `bool` | `false` | no |
 | <a name="input_availability_zone"></a> [availability\_zone](#input\_availability\_zone) | The availability zone of the instance. | `string` | `null` | no |
-| <a name="input_backup_retention_period"></a> [backup\_retention\_period](#input\_backup\_retention\_period) | The days to retain backups for. | `number` | `1` | no |
+| <a name="input_backup_retention_period"></a> [backup\_retention\_period](#input\_backup\_retention\_period) | The days to retain backups for. | `number` | `null` | no |
 | <a name="input_backup_window"></a> [backup\_window](#input\_backup\_window) | The daily time range (in UTC) during which automated backups are created if they are enabled. | `string` | `"03:00-06:00"` | no |
 | <a name="input_blue_green_update_enabled"></a> [blue\_green\_update\_enabled](#input\_blue\_green\_update\_enabled) | Enables low-downtime updates when true. | `bool` | `false` | no |
 | <a name="input_ca_cert_identifier"></a> [ca\_cert\_identifier](#input\_ca\_cert\_identifier) | The identifier of the CA certificate for the DB instance. | `string` | `null` | no |

main.tf

@@ -12,6 +12,8 @@ terraform {
 locals {
   db_subnet_group_name = var.db_subnet_group_name != null ? var.db_subnet_group_name : (length(var.subnet_ids) > 0 ? var.instance_name : null)
   parameter_group_name = var.parameter_group_name != null ? var.parameter_group_name : (length(var.parameter_group_list) > 0 ? var.instance_name : null)
+
+  backup_retention_period = var.blue_green_update_enabled ? coalesce(var.backup_retention_period, 1) : var.backup_retention_period
 }
 
 resource "aws_db_parameter_group" "main" {
@@ -47,7 +49,7 @@ resource "aws_db_instance" "main" {
   allocated_storage          = var.allocated_storage
   auto_minor_version_upgrade = var.auto_minor_version_upgrade
   availability_zone          = var.availability_zone
-  backup_retention_period    = var.backup_retention_period
+  backup_retention_period    = local.backup_retention_period
   backup_window              = var.backup_window
 
   blue_green_update {

variables.tf

@@ -19,7 +19,7 @@ variable "availability_zone" {
 variable "backup_retention_period" {
   description = "The days to retain backups for."
   type        = number
-  default     = 1
+  default     = null
 }
 
 variable "backup_window" {