feat: add security_group_name_prefix_enabled

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

Author: nitrocode

README.md

@@ -462,6 +462,7 @@ Available targets:
 | <a name="input_retention_period"></a> [retention\_period](#input\_retention\_period) | Number of days to retain backups for | `number` | `5` | no |
 | <a name="input_s3_import"></a> [s3\_import](#input\_s3\_import) | Restore from a Percona Xtrabackup in S3. The `bucket_name` is required to be in the same region as the resource. | <pre>object({<br/>    bucket_name           = string<br/>    bucket_prefix         = string<br/>    ingestion_role        = string<br/>    source_engine         = string<br/>    source_engine_version = string<br/>  })</pre> | `null` | no |
 | <a name="input_scaling_configuration"></a> [scaling\_configuration](#input\_scaling\_configuration) | List of nested attributes with scaling properties. Only valid when `engine_mode` is set to `serverless` | <pre>list(object({<br/>    auto_pause               = bool<br/>    max_capacity             = number<br/>    min_capacity             = number<br/>    seconds_until_auto_pause = number<br/>    timeout_action           = string<br/>  }))</pre> | `[]` | no |
+| <a name="input_security_group_name_prefix_enabled"></a> [security\_group\_name\_prefix\_enabled](#input\_security\_group\_name\_prefix\_enabled) | Set to `true` to use `name_prefix` to name of the security group. Set to `false` to use `name` instead | `bool` | `false` | no |
 | <a name="input_security_groups"></a> [security\_groups](#input\_security\_groups) | List of security groups to be allowed to connect to the DB instance | `list(string)` | `[]` | no |
 | <a name="input_serverlessv2_scaling_configuration"></a> [serverlessv2\_scaling\_configuration](#input\_serverlessv2\_scaling\_configuration) | serverlessv2 scaling properties | <pre>object({<br/>    min_capacity = number<br/>    max_capacity = number<br/>  })</pre> | `null` | no |
 | <a name="input_skip_final_snapshot"></a> [skip\_final\_snapshot](#input\_skip\_final\_snapshot) | Determines whether a final DB snapshot is created before the DB cluster is deleted | `bool` | `true` | no |

docs/terraform.md

@@ -148,6 +148,7 @@
 | <a name="input_retention_period"></a> [retention\_period](#input\_retention\_period) | Number of days to retain backups for | `number` | `5` | no |
 | <a name="input_s3_import"></a> [s3\_import](#input\_s3\_import) | Restore from a Percona Xtrabackup in S3. The `bucket_name` is required to be in the same region as the resource. | <pre>object({<br/>    bucket_name           = string<br/>    bucket_prefix         = string<br/>    ingestion_role        = string<br/>    source_engine         = string<br/>    source_engine_version = string<br/>  })</pre> | `null` | no |
 | <a name="input_scaling_configuration"></a> [scaling\_configuration](#input\_scaling\_configuration) | List of nested attributes with scaling properties. Only valid when `engine_mode` is set to `serverless` | <pre>list(object({<br/>    auto_pause               = bool<br/>    max_capacity             = number<br/>    min_capacity             = number<br/>    seconds_until_auto_pause = number<br/>    timeout_action           = string<br/>  }))</pre> | `[]` | no |
+| <a name="input_security_group_name_prefix_enabled"></a> [security\_group\_name\_prefix\_enabled](#input\_security\_group\_name\_prefix\_enabled) | Set to `true` to use `name_prefix` to name of the security group. Set to `false` to use `name` instead | `bool` | `false` | no |
 | <a name="input_security_groups"></a> [security\_groups](#input\_security\_groups) | List of security groups to be allowed to connect to the DB instance | `list(string)` | `[]` | no |
 | <a name="input_serverlessv2_scaling_configuration"></a> [serverlessv2\_scaling\_configuration](#input\_serverlessv2\_scaling\_configuration) | serverlessv2 scaling properties | <pre>object({<br/>    min_capacity = number<br/>    max_capacity = number<br/>  })</pre> | `null` | no |
 | <a name="input_skip_final_snapshot"></a> [skip\_final\_snapshot](#input\_skip\_final\_snapshot) | Determines whether a final DB snapshot is created before the DB cluster is deleted | `bool` | `true` | no |

examples/complete/fixtures.us-east-2.tfvars

@@ -38,3 +38,5 @@ intra_security_group_traffic_enabled = true
 parameter_group_name_prefix_enabled = true
 
 rds_cluster_identifier_prefix_enabled = true
+
+security_group_name_prefix_enabled = true

examples/complete/main.tf

@@ -48,6 +48,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
 
   cluster_parameters = [
     {

examples/complete/variables.tf

@@ -103,3 +103,10 @@ variable "rds_cluster_identifier_prefix_enabled" {
   default     = false
   description = "Set to `true` to use `identifier_prefix` to name the cluster. Set to `false` to use `identifier` instead"
 }
+
+variable "security_group_name_prefix_enabled" {
+  type        = bool
+  default     = false
+  description = "Set to `true` to use `name_prefix` to name of the security group. Set to `false` to use `name` instead"
+}
+

main.tf

@@ -24,7 +24,8 @@ data "aws_partition" "current" {
 # TODO: Use cloudposse/security-group module
 resource "aws_security_group" "default" {
   count       = local.enabled ? 1 : 0
-  name        = module.this.id
+  name_prefix = var.security_group_name_prefix_enabled ? "${module.this.id}${module.this.delimiter}" : null
+  name        = !var.security_group_name_prefix_enabled ? module.this.id : null
   description = "Allow inbound traffic from Security Groups and CIDRs"
   vpc_id      = var.vpc_id
   tags        = module.this.tags

variables.tf

@@ -565,6 +565,12 @@ variable "rds_cluster_identifier_prefix_enabled" {
   description = "Set to `true` to use `identifier_prefix` to name the cluster. Set to `false` to use `identifier` instead"
 }
 
+variable "security_group_name_prefix_enabled" {
+  type        = bool
+  default     = false
+  description = "Set to `true` to use `name_prefix` to name of the security group. Set to `false` to use `name` instead"
+}
+
 variable "enable_global_write_forwarding" {
   type        = bool
   default     = null