feat: add db subnet group prefix

Signed-off-by: nitrocode <[email protected]>

Author: nitrocode

Diff for: README.md

@@ -408,6 +408,7 @@ Available targets:
 | <a name="input_db_name"></a> [db\_name](#input\_db\_name) | Database name (default is not to create a database) | `string` | `""` | no |
 | <a name="input_db_parameter_group_name"></a> [db\_parameter\_group\_name](#input\_db\_parameter\_group\_name) | The name to give to the created `aws_db_parameter_group` resource.<br/>If omitted, the module will generate a name. | `string` | `""` | no |
 | <a name="input_db_port"></a> [db\_port](#input\_db\_port) | Database port | `number` | `3306` | no |
+| <a name="input_db_subnet_group_name_prefix_enabled"></a> [db\_subnet\_group\_name\_prefix\_enabled](#input\_db\_subnet\_group\_name\_prefix\_enabled) | Set to `true` to use `name_prefix` to name of the DB subnet group name. Set to `false` to use `name` instead | `bool` | `false` | no |
 | <a name="input_deletion_protection"></a> [deletion\_protection](#input\_deletion\_protection) | If the DB instance should have deletion protection enabled | `bool` | `false` | no |
 | <a name="input_delimiter"></a> [delimiter](#input\_delimiter) | Delimiter to be used between ID elements.<br/>Defaults to `-` (hyphen). Set to `""` to use no delimiter at all. | `string` | `null` | no |
 | <a name="input_descriptor_formats"></a> [descriptor\_formats](#input\_descriptor\_formats) | Describe additional descriptors to be output in the `descriptors` output map.<br/>Map of maps. Keys are names of descriptors. Values are maps of the form<br/>`{<br/>   format = string<br/>   labels = list(string)<br/>}`<br/>(Type is `any` so the map values can later be enhanced to provide additional options.)<br/>`format` is a Terraform format string to be passed to the `format()` function.<br/>`labels` is a list of labels, in order, to pass to `format()` function.<br/>Label values will be normalized before being passed to `format()` so they will be<br/>identical to how they appear in `id`.<br/>Default is `{}` (`descriptors` output will be empty). | `any` | `{}` | no |

Diff for: docs/terraform.md

@@ -94,6 +94,7 @@
 | <a name="input_db_name"></a> [db\_name](#input\_db\_name) | Database name (default is not to create a database) | `string` | `""` | no |
 | <a name="input_db_parameter_group_name"></a> [db\_parameter\_group\_name](#input\_db\_parameter\_group\_name) | The name to give to the created `aws_db_parameter_group` resource.<br/>If omitted, the module will generate a name. | `string` | `""` | no |
 | <a name="input_db_port"></a> [db\_port](#input\_db\_port) | Database port | `number` | `3306` | no |
+| <a name="input_db_subnet_group_name_prefix_enabled"></a> [db\_subnet\_group\_name\_prefix\_enabled](#input\_db\_subnet\_group\_name\_prefix\_enabled) | Set to `true` to use `name_prefix` to name of the DB subnet group name. Set to `false` to use `name` instead | `bool` | `false` | no |
 | <a name="input_deletion_protection"></a> [deletion\_protection](#input\_deletion\_protection) | If the DB instance should have deletion protection enabled | `bool` | `false` | no |
 | <a name="input_delimiter"></a> [delimiter](#input\_delimiter) | Delimiter to be used between ID elements.<br/>Defaults to `-` (hyphen). Set to `""` to use no delimiter at all. | `string` | `null` | no |
 | <a name="input_descriptor_formats"></a> [descriptor\_formats](#input\_descriptor\_formats) | Describe additional descriptors to be output in the `descriptors` output map.<br/>Map of maps. Keys are names of descriptors. Values are maps of the form<br/>`{<br/>   format = string<br/>   labels = list(string)<br/>}`<br/>(Type is `any` so the map values can later be enhanced to provide additional options.)<br/>`format` is a Terraform format string to be passed to the `format()` function.<br/>`labels` is a list of labels, in order, to pass to `format()` function.<br/>Label values will be normalized before being passed to `format()` so they will be<br/>identical to how they appear in `id`.<br/>Default is `{}` (`descriptors` output will be empty). | `any` | `{}` | no |

Diff for: examples/complete/fixtures.us-east-2.tfvars

@@ -40,3 +40,5 @@ parameter_group_name_prefix_enabled = true
 rds_cluster_identifier_prefix_enabled = true
 
 security_group_name_prefix_enabled = true
+
+db_subnet_group_name_prefix_enabled = true

Diff for: examples/complete/main.tf

@@ -49,6 +49,7 @@ module "rds_cluster" {
   parameter_group_name_prefix_enabled   = var.parameter_group_name_prefix_enabled
   rds_cluster_identifier_prefix_enabled = var.rds_cluster_identifier_prefix_enabled
   security_group_name_prefix_enabled    = var.security_group_name_prefix_enabled
+  db_subnet_group_name_prefix_enabled   = var.db_subnet_group_name_prefix_enabled
 
   cluster_parameters = [
     {

Diff for: examples/complete/variables.tf

@@ -110,3 +110,8 @@ variable "security_group_name_prefix_enabled" {
   description = "Set to `true` to use `name_prefix` to name of the security group. Set to `false` to use `name` instead"
 }
 
+variable "db_subnet_group_name_prefix_enabled" {
+  type        = bool
+  default     = false
+  description = "Set to `true` to use `name_prefix` to name of the DB subnet group name. Set to `false` to use `name` instead"
+}

Diff for: main.tf

@@ -405,7 +405,10 @@ resource "aws_rds_cluster_instance" "default" {
 
 resource "aws_db_subnet_group" "default" {
   count       = local.enabled ? 1 : 0
-  name        = try(length(var.subnet_group_name), 0) == 0 ? module.this.id : var.subnet_group_name
+
+  name_prefix = var.db_subnet_group_name_prefix_enabled ? (try(length(var.subnet_group_name), 0) == 0 ? "${module.this.id}${module.delimiter.id}" : var.subnet_group_name) : null
+  name        = !var.db_subnet_group_name_prefix_enabled ? (try(length(var.subnet_group_name), 0) == 0 ? module.this.id : var.subnet_group_name) : null
+
   description = "Allowed subnets for DB cluster instances"
   subnet_ids  = var.subnets
   tags        = module.this.tags

Diff for: variables.tf

@@ -571,6 +571,12 @@ variable "security_group_name_prefix_enabled" {
   description = "Set to `true` to use `name_prefix` to name of the security group. Set to `false` to use `name` instead"
 }
 
+variable "db_subnet_group_name_prefix_enabled" {
+  type        = bool
+  default     = false
+  description = "Set to `true` to use `name_prefix` to name of the DB subnet group name. Set to `false` to use `name` instead"
+}
+
 variable "enable_global_write_forwarding" {
   type        = bool
   default     = null