-
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
DocumentDB: Added aws_docdb_cluster_instance resource #7143
Conversation
1ac6273
to
22aeb99
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @fewstera this is ready for rebase and provided an initial review below. All pretty small stuff so should be fairly easy to get in. 😄 Thanks for these contributions!
22aeb99
to
21c90c8
Compare
@bflad I think I've made all the changes requested. I haven't had the chance to run the acceptance tests yet, I'm about to go out, so can't run them now, I will post the output when I get the chance. Thanks again for checking the PR so thoroughly. 👍 |
b2ff187
to
e01c760
Compare
@bflad acceptance tests now passing.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are going to make a bunch of folks happy, @fewstera! LGTM. 🚀
--- PASS: TestAccAWSDocDBClusterInstance_disappears (658.86s)
--- PASS: TestAccAWSDocDBClusterInstance_namePrefix (666.71s)
--- PASS: TestAccAWSDocDBClusterInstance_az (694.22s)
--- PASS: TestAccAWSDocDBClusterInstance_kmsKey (702.83s)
--- PASS: TestAccAWSDocDBClusterInstance_generatedName (747.87s)
--- PASS: TestAccAWSDocDBClusterInstance_basic (1324.65s)
This has been released in version 1.59.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. |
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! |
This depends on my previous two PRs for DocumentDB (#7122 and #7123 ) so must be merged after those.
Forms part of #7077.