Skip to content

Commit

Permalink
fix: Expose externalized redis to helm for console metrics (#341)
Browse files Browse the repository at this point in the history
* fix: Expose externalized redis to helm for console metrics

* tf lint fix

* INFRA-549 revert to wandb redis install->false

* INFRA-549 move redis.external to globalˆ

* tf lint
  • Loading branch information
j7m4 authored Feb 25, 2025
1 parent f1f6277 commit c14dc28
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -303,8 +303,9 @@ module "wandb" {
}

redis = {
host = var.use_external_redis ? var.external_redis_host : (var.create_elasticache ? module.redis[0].host : "")
port = var.use_external_redis ? var.external_redis_port : (var.create_elasticache ? "${module.redis[0].port}?tls=true&ttlInSeconds=604800" : "")
host = var.use_external_redis ? var.external_redis_host : (var.create_elasticache ? module.redis[0].host : "")
port = var.use_external_redis ? var.external_redis_port : (var.create_elasticache ? "${module.redis[0].port}?tls=true&ttlInSeconds=604800" : "")
external = var.use_external_redis
}
}

Expand Down

0 comments on commit c14dc28

Please sign in to comment.