cloudposse / terraform-aws-elasticache-redis

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

Fix cloudwatch alarm failure to create (#56) #77

Closed varunpalekar closed 4 years ago

varunpalekar commented 4 years ago

what

why

references

osterman commented 4 years ago

/test all

joe-niland commented 4 years ago

/terraform-fmt

joe-niland commented 4 years ago

/test all

joe-niland commented 4 years ago

@varunpalekar please have a look at this issue causing the tests to fail:

image

varunpalekar commented 4 years ago

/test all

varunpalekar commented 4 years ago

@joe-niland Thanks for the review. I updated the code respectively. I also find an issue when running on a new infra, also fixed that.

joe-niland commented 4 years ago

/test all

joe-niland commented 4 years ago

Thanks for your contribution @varunpalekar. Released as https://github.com/cloudposse/terraform-aws-elasticache-redis/releases/tag/0.23.0