-
Notifications
You must be signed in to change notification settings - Fork 9.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RDS enabled_cloudwatch_logs_exports - InvalidParameterCombination: No modifications were requested #6799
Comments
…s_exports arguments Reference: #6799 Changes: ``` * resource/aws_db_instance: Prevent ordering differences with `enabled_cloudwatch_logs_exports` argument * resource/aws_rds_cluster: Prevent ordering differences with `enabled_cloudwatch_logs_exports` argument ``` Output from acceptance testing: ``` --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Postgresql (552.53s) --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MSSQL (742.59s) --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Oracle (805.86s) --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MySQL (828.42s) --- PASS: TestAccAWSRDSCluster_EnabledCloudwatchLogsExports_Postgresql (133.14s) --- PASS: TestAccAWSRDSCluster_EnabledCloudwatchLogsExports_MySQL (243.49s) ```
…s_exports arguments (#15404) * service/rds: Prevent ordering differences with enabled_cloudwatch_logs_exports arguments Reference: #6799 Changes: ``` * resource/aws_db_instance: Prevent ordering differences with `enabled_cloudwatch_logs_exports` argument * resource/aws_rds_cluster: Prevent ordering differences with `enabled_cloudwatch_logs_exports` argument ``` Output from acceptance testing: ``` --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Postgresql (552.53s) --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MSSQL (742.59s) --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Oracle (805.86s) --- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MySQL (828.42s) --- PASS: TestAccAWSRDSCluster_EnabledCloudwatchLogsExports_Postgresql (133.14s) --- PASS: TestAccAWSRDSCluster_EnabledCloudwatchLogsExports_MySQL (243.49s) ``` * docs/service/rds: List -> Set for enabled_cloudwatch_logs_exports arguments
The fix for this has been merged and will release with version 3.9.0 of the Terraform AWS Provider, likely tomorrow. 👍 |
This has been released in version 3.9.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Terraform Version
Terraform v0.11.10
Affected Resource(s)
aws_db_instance
Terraform Configuration Files
RDS instance with the below attribute configuration.
Debug Output
Expected Behavior
On subsequent runs it shouldn't detect any changes.
Actual Behavior
If I re-run the plan - it keeps saying that there is an update required.
See the plan output - it just tries to change order.
Steps to Reproduce
run
terraform apply
on a RDS instance withenabled_cloudwatch_logs_exports
enabled and then try to re-run the plan agin.Important Factoids
It looks like the ordering in
enabled_cloudwatch_logs_exports
is important.If I set it like this it works fine.
Note: The documentation doesn't mention that the list should be alphabetically ordered to work properly.
The text was updated successfully, but these errors were encountered: