From c6b60528888e964de7f027fff1af0b52f55b4156 Mon Sep 17 00:00:00 2001
From: Michael Barrientos <mbarrientos@chanzuckerberg.com>
Date: Thu, 10 Oct 2019 01:22:51 -0700
Subject: [PATCH] Make aws-redis-node match internal lib, upgrade default

---
 aws-redis-node/main.tf      | 2 +-
 aws-redis-node/variables.tf | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/aws-redis-node/main.tf b/aws-redis-node/main.tf
index e494427d..0507a1f2 100755
--- a/aws-redis-node/main.tf
+++ b/aws-redis-node/main.tf
@@ -19,7 +19,7 @@ resource "aws_elasticache_subnet_group" "default" {
 resource "aws_elasticache_cluster" "default" {
   cluster_id           = "${var.resource_name != "" ? var.resource_name : local.name}"
   engine               = "redis"
-  engine_version       = "3.2.6"
+  engine_version       = "${var.engine_version}"
   node_type            = "${var.instance_type}"
   port                 = "${var.port}"
   num_cache_nodes      = 1
diff --git a/aws-redis-node/variables.tf b/aws-redis-node/variables.tf
index 1a4c4a7f..bb91183a 100755
--- a/aws-redis-node/variables.tf
+++ b/aws-redis-node/variables.tf
@@ -21,6 +21,7 @@ variable "subnets" {
 variable "availability_zone" {
   type        = "string"
   description = "Availability zone in which this instance should run."
+  default     = null
 }
 
 variable "ingress_security_group_ids" {
@@ -42,17 +43,17 @@ variable "port" {
 variable "instance_type" {
   type        = "string"
   description = "The type of instance to run. See [supported node types](https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/CacheNodes.SupportedTypes.html)"
-  default     = "cache.m4.large"
+  default     = "cache.m5.large"
 }
 
 variable "parameter_group_name" {
-  default = "default.redis3.2"
+  default = "default.redis5.0"
 }
 
 variable "engine_version" {
   type        = "string"
   description = "The version of Redis to run. See [supported versions](https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/supported-engine-versions.html)"
-  default     = "4.0.10"
+  default     = "5.0.5"
 }
 
 variable "apply_immediately" {