search

cloudposse / terraform-aws-elasticache-redis

Terraform module to provision an ElastiCache Redis Cluster
https://cloudposse.com/accelerate
Apache License 2.0
143 stars 248 forks source link

feat: Add support for Valkey engine #249

Closed adamantike closed 1 month ago

adamantike commented 1 month ago

what

This change allows setting the engine variable, to allow using Valkey as the cache engine.

No breaking changes are expected, as the default value for the engine variable is set to redis. Also, the minimum AWS provider version is not changed as using Redis does not require using a newer provider.

references

nitrocode commented 1 month ago

/terratest

github-actions[bot] commented 1 month ago

These changes were released in v1.7.0.

Nuru commented 4 weeks ago

@nitrocode In the future, before approving a PR like this, please ensure that enumerators like these (engine is either redis or valkey) have their legal values enumerated along with explanations of what they mean. Short explanation (e.g. "type of cache") in the variable description; long explanation with link to even more information in the README.