Skip to content

T
terraform-aws-rds
Commit 470077ea authored by Fernanda Martins's avatar Fernanda Martins Committed by Anton Babenko
Browse files
Options

Fix for issue when MonitoringRoleARN is required (#143)

parent e015fa3a
Show whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
modules/db_instance/main.tf
View file @ 470077ea
......@@ -70,7 +70,7 @@ resource "aws_db_instance" "this" {
iops = var.iops
publicly_accessible = var.publicly_accessible
monitoring_interval = var.monitoring_interval
monitoring_role_arn = coalesce(var.monitoring_role_arn, aws_iam_role.enhanced_monitoring.*.arn, null)
monitoring_role_arn = var.monitoring_interval > 0 ? coalesce(var.monitoring_role_arn, join(", ", aws_iam_role.enhanced_monitoring.*.arn), null) : null
allow_major_version_upgrade = var.allow_major_version_upgrade
auto_minor_version_upgrade = var.auto_minor_version_upgrade
......@@ -143,7 +143,7 @@ resource "aws_db_instance" "this_mssql" {
iops = var.iops
publicly_accessible = var.publicly_accessible
monitoring_interval = var.monitoring_interval
monitoring_role_arn = coalesce(var.monitoring_role_arn, aws_iam_role.enhanced_monitoring.*.arn, null)
monitoring_role_arn = var.monitoring_interval > 0 ? coalesce(var.monitoring_role_arn, aws_iam_role.enhanced_monitoring.*.arn, null) : null
allow_major_version_upgrade = var.allow_major_version_upgrade
auto_minor_version_upgrade = var.auto_minor_version_upgrade
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment