Skip to content

Commit 24cfe48

Browse files
author
Abdul Wahid
authored
Fixing conditions (#18)
* fix: assignment compatiblity issues * chore: documentation
1 parent 5c07978 commit 24cfe48

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

CHANGELOG.md

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,13 @@ All notable changes to this project will be documented in this file.
55
<a name="unreleased"></a>
66
## [Unreleased]
77

8-
- Upgrade to 5.x compatibility + update examples/variables
8+
- fix: assignment compatiblity issues
9+
10+
11+
<a name="3.3.0"></a>
12+
## [3.3.0] - 2023-07-27
13+
14+
- Upgrade to 5.x compatibility + update examples/variables ([#17](https://github.com/umotif-public/terraform-aws-rds-aurora/issues/17))
915

1016

1117
<a name="3.2.1"></a>
@@ -109,7 +115,8 @@ All notable changes to this project will be documented in this file.
109115
- Initial commit
110116

111117

112-
[Unreleased]: https://github.com/umotif-public/terraform-aws-rds-aurora/compare/3.2.1...HEAD
118+
[Unreleased]: https://github.com/umotif-public/terraform-aws-rds-aurora/compare/3.3.0...HEAD
119+
[3.3.0]: https://github.com/umotif-public/terraform-aws-rds-aurora/compare/3.2.1...3.3.0
113120
[3.2.1]: https://github.com/umotif-public/terraform-aws-rds-aurora/compare/3.2.0...3.2.1
114121
[3.2.0]: https://github.com/umotif-public/terraform-aws-rds-aurora/compare/3.1.0...3.2.0
115122
[3.1.0]: https://github.com/umotif-public/terraform-aws-rds-aurora/compare/3.0.0...3.1.0

main.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ resource "aws_rds_cluster" "main" {
128128

129129
port = var.port == "" ? var.engine == "aurora-postgresql" ? "5432" : "3306" : var.port
130130
db_subnet_group_name = var.db_subnet_group_name == "" ? aws_db_subnet_group.main[0].name : var.db_subnet_group_name
131-
vpc_security_group_ids = compact(concat(aws_security_group.main[0].id, var.vpc_security_group_ids))
131+
vpc_security_group_ids = compact(concat([aws_security_group.main[0].id], var.vpc_security_group_ids))
132132
storage_encrypted = var.storage_encrypted
133133

134134
db_cluster_parameter_group_name = var.create_parameter_group ? aws_rds_cluster_parameter_group.main[0].id : var.db_cluster_parameter_group_name
@@ -298,7 +298,7 @@ resource "aws_rds_cluster_instance" "main" {
298298
preferred_maintenance_window = var.preferred_instance_maintenance_window
299299
apply_immediately = var.apply_immediately
300300

301-
monitoring_role_arn = var.create_monitoring_role ? aws_iam_role.rds_enhanced_monitoring[0].arn : var.monitoring_role_arn
301+
monitoring_role_arn = var.create_monitoring_role && var.monitoring_interval > 0 ? aws_iam_role.rds_enhanced_monitoring[0].arn : var.monitoring_role_arn
302302
monitoring_interval = var.monitoring_interval
303303
auto_minor_version_upgrade = var.auto_minor_version_upgrade
304304
performance_insights_enabled = var.performance_insights_enabled

0 commit comments

Comments
 (0)