fix: Var name (#8)

README.md

@@ -183,7 +183,7 @@ To-do:
 | <a name="input_description_redis"></a> [description\_redis](#input\_description\_redis) | User-created description for the replication group. Must not be empty | `string` | `""` | no |
 | <a name="input_description_serverless"></a> [description\_serverless](#input\_description\_serverless) | User-created description for the serverless. Must not be empty | `string` | `""` | no |
 | <a name="input_description_subnet"></a> [description\_subnet](#input\_description\_subnet) | The Description of the ElastiCache Subnet Group. | `string` | `null` | no |
-| <a name="input_enabled_limits"></a> [enabled\_limits](#input\_enabled\_limits) | bla | `bool` | `false` | no |
+| <a name="input_enabled_limits"></a> [enabled\_limits](#input\_enabled\_limits) | Enable customization of cache\_usage\_limits(Only for the Serveless module) | `bool` | `false` | no |
 | <a name="input_engine"></a> [engine](#input\_engine) | Name of the cache engine to be used for the clusters in this replication group. | `string` | `"redis"` | no |
 | <a name="input_engine_version"></a> [engine\_version](#input\_engine\_version) | Version number of the cache engine to be used for the cache clusters in this replication group. | `string` | `"6.x"` | no |
 | <a name="input_family"></a> [family](#input\_family) | The family of the ElastiCache parameter group. | `string` | `"redis6.x"` | no |
@@ -194,7 +194,6 @@ To-do:
 | <a name="input_maintenance_window"></a> [maintenance\_window](#input\_maintenance\_window) | Specifies the weekly time range for when maintenance on the cache cluster is performed. | `string` | `"sat:03:00-sat:06:00"` | no |
 | <a name="input_major_engine_version"></a> [major\_engine\_version](#input\_major\_engine\_version) | The version of the cache engine that will be used to create the serverless cache.(Only for the Serveless module) | `string` | `"7"` | no |
 | <a name="input_multi_az_enabled"></a> [multi\_az\_enabled](#input\_multi\_az\_enabled) | Specifies whether to enable Multi-AZ Support for the replication group. | `bool` | `false` | no |
-| <a name="input_name_serverless"></a> [name\_serverless](#input\_name\_serverless) | Name Redis Serverless. This parameter is stored as a lowercase string. | `string` | `null` | no |
 | <a name="input_node_type"></a> [node\_type](#input\_node\_type) | Instance class to be used. | `string` | `null` | no |
 | <a name="input_notification_topic_arn"></a> [notification\_topic\_arn](#input\_notification\_topic\_arn) | ARN of an SNS topic to send ElastiCache notifications to | `string` | `null` | no |
 | <a name="input_num_cache_clusters"></a> [num\_cache\_clusters](#input\_num\_cache\_clusters) | Number of cache clusters (primary and replicas) this replication group will have. | `number` | `3` | no |
@@ -210,6 +209,7 @@ To-do:
 | <a name="input_replication_group_id"></a> [replication\_group\_id](#input\_replication\_group\_id) | Replication group identifier. This parameter is stored as a lowercase string. | `string` | `null` | no |
 | <a name="input_security_group_ids"></a> [security\_group\_ids](#input\_security\_group\_ids) | One or more Amazon VPC security groups associated with this replication group. | `list(string)` | `[]` | no |
 | <a name="input_security_group_names"></a> [security\_group\_names](#input\_security\_group\_names) | List of cache security group names to associate with this replication group. | `list(string)` | `null` | no |
+| <a name="input_serverless_name"></a> [serverless\_name](#input\_serverless\_name) | Name Redis Serverless. This parameter is stored as a lowercase string. | `string` | `null` | no |
 | <a name="input_snapshot_arns_to_restore"></a> [snapshot\_arns\_to\_restore](#input\_snapshot\_arns\_to\_restore) | The list of ARN(s) of the snapshot that the new serverless cache will be created from(Only for the Serveless module) | `list(string)` | `null` | no |
 | <a name="input_snapshot_name"></a> [snapshot\_name](#input\_snapshot\_name) | Name of a snapshot from which to restore data into the new node group. | `string` | `null` | no |
 | <a name="input_snapshot_retention_limit"></a> [snapshot\_retention\_limit](#input\_snapshot\_retention\_limit) | Number of days for which ElastiCache will retain automatic cache cluster snapshots before deleting them. | `number` | `7` | no |

main.tf

@@ -75,7 +75,7 @@ module "elasticache_parameter_group" {
 
   create = var.create_elasticache_parameter_group
 
-  name        = coalesce(var.parameter_group_name, var.replication_group_id)
+  name        = try(coalesce(var.parameter_group_name, var.replication_group_id), null)
   family      = var.family
   description = var.description_parameter
   parameters  = var.parameters
@@ -121,7 +121,7 @@ resource "aws_elasticache_serverless_cache" "this" {
   count = var.create_elasticache_serveless ? 1 : 0
 
   engine                   = var.engine
-  name                     = lower(var.name_serverless)
+  name                     = lower(var.serverless_name)
   description              = var.description_serverless
   daily_snapshot_time      = var.daily_snapshot_time
   kms_key_id               = var.kms_key_id

variables.tf

@@ -51,7 +51,7 @@ variable "create_cache_password" {
 }
 ##############################################################################################
 #Redis Serveless
-variable "name_serverless" {
+variable "serverless_name" {
   type        = string
   default     = null
   description = "Name Redis Serverless. This parameter is stored as a lowercase string."
@@ -78,7 +78,7 @@ variable "cache_usage_limits" {
 variable "enabled_limits" {
   type        = bool
   default     = false
-  description = "bla"
+  description = "Enable customization of cache_usage_limits(Only for the Serveless module)"
 }
 
 variable "major_engine_version" {