luciano/terraform-aws-rds-proxy
1
0
Fork 0
mirror of https://github.com/terraform-aws-modules/terraform-aws-rds-proxy.git synced 2025-12-17 00:11:11 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove the un-necessary local.

Browse source 
Instead add the conditional to the `for_each` statement.
Thanks for the suggestion @bryantbiggs.
This commit is contained in:
Gavin Williams 2022-08-10 17:29:26 +01:00
parent d72ae6a5da
commit 2a76143c5c
No known key found for this signature in database
GPG key ID: 6D72FC116961D41D
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
main.tf
View file

@ -2,7 +2,6 @@ locals {
role_arn = var.create_proxy && var.create_iam_role ? aws_iam_role.this[0].arn : var.role_arn role_arn = var.create_proxy && var.create_iam_role ? aws_iam_role.this[0].arn : var.role_arn
role_name = coalesce(var.iam_role_name, var.name) role_name = coalesce(var.iam_role_name, var.name)
policy_name = coalesce(var.iam_policy_name, var.name) policy_name = coalesce(var.iam_policy_name, var.name)
db_proxy_endpoints = var.create_proxy ? var.db_proxy_endpoints : {}
} }
data "aws_region" "current" {} data "aws_region" "current" {}
@ -69,7 +68,7 @@ resource "aws_db_proxy_target" "db_cluster" {
} }
resource "aws_db_proxy_endpoint" "this" { resource "aws_db_proxy_endpoint" "this" {
for_each = local.db_proxy_endpoints for_each = { for k, v in var.db_proxy_endpoints : k => v if var.create_proxy }
db_proxy_name = aws_db_proxy.this[0].name db_proxy_name = aws_db_proxy.this[0].name
db_proxy_endpoint_name = each.value.name db_proxy_endpoint_name = each.value.name