-
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
New resource: Timestream Write database resource #15463
New resource: Timestream Write database resource #15463
Conversation
This PR is 2 out of 2 PRs, split from #15444 per @DrFaust92 request |
Also, this is my first ever Go code I wrote. |
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 @yardensachs,
Some comments
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.
small nit, otherwise looking good. ill let maintainers take over for now as I cant run tests to verify at the moment
@bflad How can I get someone to run tests so I can merge? |
ok, Im assuming its a matter of time till it does |
50bf7fb
to
5e45786
Compare
@DrFaust92 thanks, do you know how long does it usually take for these types of PRs to be reviewed? |
8418998
to
d6e67b6
Compare
@DrFaust92 Thanks for explaining. So does this mean that this PR potentially can wait months to be merged? |
d6e67b6
to
3737723
Compare
3737723
to
a870d28
Compare
45fb494
to
5aa9e3e
Compare
Hi @yardensachs , thank you for this PR! Overall looked really great, I've only refactored a bit to update to using new Terraform Plugin SDK functions like Output of acceptance tests (commercial):
Output of acceptance tests (GovCloud):
|
5aa9e3e
to
27fa37d
Compare
27fa37d
to
13f913d
Compare
This has been released in version 3.40.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 pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Community Note
Closes #15421
Release note for CHANGELOG:
Output from acceptance testing: