Skip to content

Commit fd3c8a0

Browse files
committed
Remove local.create_cluster logic due to fixed provider bug
Provider bug was fixed in 5.84, but version is already bumped to 5.93, so no update needed. Fix: #43
1 parent d10bc36 commit fd3c8a0

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

main.tf

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,6 @@ locals {
22
# https://github.com/hashicorp/terraform-provider-aws/blob/3c4cb52c5dc2c09e10e5a717f73d1d8bc4186e87/internal/service/elasticache/cluster.go#L271
33
in_replication_group = var.replication_group_id != null
44

5-
# elasticache clusters currently do not support engine type valkey
6-
# TODO: remove this local `create_cluster` conditional once this bug is addressed:
7-
# https://github.com/hashicorp/terraform-provider-aws/issues/39905
8-
create_cluster = var.create_cluster && var.engine != "valkey" ? true : false
9-
105
security_group_ids = local.create_security_group ? concat(var.security_group_ids, [aws_security_group.this[0].id]) : var.security_group_ids
116
port = var.engine == "memcached" ? 11211 : 6379
127

@@ -18,7 +13,7 @@ locals {
1813
################################################################################
1914

2015
resource "aws_elasticache_cluster" "this" {
21-
count = var.create && local.create_cluster ? 1 : 0
16+
count = var.create && var.create_cluster ? 1 : 0
2217

2318
apply_immediately = var.apply_immediately
2419
auto_minor_version_upgrade = var.auto_minor_version_upgrade

0 commit comments

Comments
 (0)