From d57dab7f8e5e06e4effaf893b1be27fe339c7eea Mon Sep 17 00:00:00 2001 From: Anton Kaymakchi <32177073+StupidScience@users.noreply.github.com> Date: Wed, 20 May 2020 23:50:56 +0300 Subject: [PATCH] azurerm_redis_cache: support new maxmemory policies `allkeys-lfu' & 'volatile-lfu` (#7031) Fixes #7030 --- azurerm/internal/services/redis/resource_arm_redis_cache.go | 4 +++- azurerm/internal/services/redis/validation_test.go | 2 ++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/azurerm/internal/services/redis/resource_arm_redis_cache.go b/azurerm/internal/services/redis/resource_arm_redis_cache.go index 0738a86250c9..ec58cdbc46a0 100644 --- a/azurerm/internal/services/redis/resource_arm_redis_cache.go +++ b/azurerm/internal/services/redis/resource_arm_redis_cache.go @@ -900,10 +900,12 @@ func validateRedisMaxMemoryPolicy(v interface{}, _ string) (warnings []string, e "allkeys-random": true, "volatile-random": true, "volatile-ttl": true, + "allkeys-lfu": true, + "volatile-lfu": true, } if !families[value] { - errors = append(errors, fmt.Errorf("Redis Max Memory Policy can only be 'noeviction' / 'allkeys-lru' / 'volatile-lru' / 'allkeys-random' / 'volatile-random' / 'volatile-ttl'")) + errors = append(errors, fmt.Errorf("Redis Max Memory Policy can only be 'noeviction' / 'allkeys-lru' / 'volatile-lru' / 'allkeys-random' / 'volatile-random' / 'volatile-ttl' / 'allkeys-lfu' / 'volatile-lfu'")) } return warnings, errors diff --git a/azurerm/internal/services/redis/validation_test.go b/azurerm/internal/services/redis/validation_test.go index 769c5e58ea15..f11b8caf08ac 100644 --- a/azurerm/internal/services/redis/validation_test.go +++ b/azurerm/internal/services/redis/validation_test.go @@ -57,6 +57,8 @@ func TestAccAzureRMRedisCacheMaxMemoryPolicy_validation(t *testing.T) { {Value: "allkeys-random", ErrCount: 0}, {Value: "volatile-random", ErrCount: 0}, {Value: "volatile-ttl", ErrCount: 0}, + {Value: "allkeys-lfu", ErrCount: 0}, + {Value: "volatile-lfu", ErrCount: 0}, {Value: "something-else", ErrCount: 1}, }