Lowercase REDIS

[ahmedfourti]

Description

Hello guys,
In the default variable default_user we can specify the engine; but by default it's set to REDIS on uppercase which creates a warning during plan

Warning: Case Insensitive Matching Deprecated
│ 
│   with module.amb.module.elasticache_user_group.aws_elasticache_user.this["amb"],
│   on .terraform/modules/amb.elasticache_user_group/modules/user-group/main.tf line 63, in resource "aws_elasticache_user" "this":
│   63:   engine               = try(each.value.engine, "REDIS")
│ 
│ Expected an exact match to "redis", got "REDIS". Case insensitive matching is deprecated for this argument. Update the value to an exact match to suppress this warning and avoid breaking changes in a future major version.

I saw this PR merged few months ago, but it looks like you missed this one.

Versions

• Module version [Required]:

• Terraform version:
  Terraform v1.13.4

• Provider version(s):
  registry.terraform.io/hashicorp/aws v6.16.0

Reproduction Code [Required]

Steps to reproduce the behavior:

module "elasticache_user_group" {
	source = "terraform-aws-modules/elasticache/aws//modules/user-group"
	version = "1.10.2"

	user_group_id       = var.general_information.product_name
	create_default_user = false
	engine = "redis"


	users = {
		"${var.general_information.product_name}" = {
			access_string = "on ~session-${var.general_information.product_name}:* +@all"
			passwords     = //my_password
			
		}
	}

	tags = merge(
		var.general_information.tags,
		var.cache_details.tags
	)
}