rds: db monitoring role to be created by TF#6
Conversation
terraform/modules/database/main.tf
Outdated
There was a problem hiding this comment.
The ARN is not required when the previous two parameters are supplied. It will create a role and then use that ARN.
There was a problem hiding this comment.
I thought so too, it actually errors out when you don't provide it
There was a problem hiding this comment.
InvalidParameterCombination: A MonitoringRoleARN value is required if you specify a MonitoringInterval value other than 0.
Feels like some parameter validation enforcement.
There was a problem hiding this comment.
looks like its supposed to be fixed in https://github.com/terraform-aws-modules/terraform-aws-rds/releases/tag/v2.12.0
we're using 2.4.0, maybe going to ^2.12.0 will solve this
There was a problem hiding this comment.
Yeah let's bump that version up!
39e73a9 to
8433cb7
Compare
RDS creation does not guarentee the enhance_monitoring role is created before TF tries to use the role, creating it will stop it from erroring
8433cb7 to
bb80f5e
Compare
rds: db monitoring role to be created by TF
RDS creation does not guarentee the enhance_monitoring role is created
before TF tries to use the role, creating it will stop it from erroring