Skip to content
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

Merged

Conversation

yardensachs
Copy link
Contributor

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Closes #15421

Release note for CHANGELOG:

FEATURES:
- **New Resource:** `aws_timestreamwrite_database`

Output from acceptance testing:

make testacc TESTARGS='-run=TestAccAWSTimestreamWriteDatabase_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSTimestreamWriteDatabase_ -timeout 120m
=== RUN   TestAccAWSTimestreamWriteDatabase_basic
=== PAUSE TestAccAWSTimestreamWriteDatabase_basic
=== RUN   TestAccAWSTimestreamWriteDatabase_kmsKey
=== PAUSE TestAccAWSTimestreamWriteDatabase_kmsKey
=== RUN   TestAccAWSTimestreamWriteDatabase_Tags
=== PAUSE TestAccAWSTimestreamWriteDatabase_Tags
=== CONT  TestAccAWSTimestreamWriteDatabase_basic
=== CONT  TestAccAWSTimestreamWriteDatabase_Tags
=== CONT  TestAccAWSTimestreamWriteDatabase_kmsKey
--- PASS: TestAccAWSTimestreamWriteDatabase_basic (17.00s)
--- PASS: TestAccAWSTimestreamWriteDatabase_kmsKey (18.86s)
--- PASS: TestAccAWSTimestreamWriteDatabase_Tags (40.68s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	43.229s

@yardensachs yardensachs requested a review from a team October 2, 2020 20:51
@ghost ghost added size/XL Managed by automation to categorize the size of a PR. documentation Introduces or discusses updates to documentation. provider Pertains to the provider itself, rather than any interaction with AWS. service/timestreamwrite Issues and PRs that pertain to the timestreamwrite service. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. labels Oct 2, 2020
@github-actions github-actions bot added the needs-triage Waiting for first response or review from a maintainer. label Oct 2, 2020
@yardensachs yardensachs changed the title F timestreamwrite database resource New resource: Timestream Write database resource Oct 2, 2020
@yardensachs
Copy link
Contributor Author

This PR is 2 out of 2 PRs, split from #15444 per @DrFaust92 request
Earlier PR was #15451

@yardensachs
Copy link
Contributor Author

Also, this is my first ever Go code I wrote.
Please be gentle 😇

Copy link
Collaborator

@DrFaust92 DrFaust92 left a 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

aws/resource_aws_timestreamwrite_database.go Outdated Show resolved Hide resolved
aws/resource_aws_timestreamwrite_database.go Outdated Show resolved Hide resolved
aws/resource_aws_timestreamwrite_database.go Show resolved Hide resolved
aws/resource_aws_timestreamwrite_database_test.go Outdated Show resolved Hide resolved
aws/resource_aws_timestreamwrite_database_test.go Outdated Show resolved Hide resolved
website/docs/r/timestreamwrite_database.html.markdown Outdated Show resolved Hide resolved
website/docs/r/timestreamwrite_database.html.markdown Outdated Show resolved Hide resolved
@ewbankkit ewbankkit removed the needs-triage Waiting for first response or review from a maintainer. label Oct 2, 2020
@yardensachs yardensachs requested review from DrFaust92 and removed request for a team October 2, 2020 22:30
Copy link
Collaborator

@DrFaust92 DrFaust92 left a 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

aws/resource_aws_timestreamwrite_database.go Outdated Show resolved Hide resolved
@yardensachs
Copy link
Contributor Author

@bflad How can I get someone to run tests so I can merge?
Is it just a time factor until someone does it?

@yardensachs
Copy link
Contributor Author

yardensachs commented Oct 5, 2020

ok, Im assuming its a matter of time till it does

@yardensachs yardensachs force-pushed the f-timestreamwrite-database-resource branch from 50bf7fb to 5e45786 Compare October 6, 2020 15:46
@yardensachs
Copy link
Contributor Author

@DrFaust92 thanks, do you know how long does it usually take for these types of PRs to be reviewed?
I don't mind, just don't want to miss a review so I keep refreshing every few hours 😅

@DrFaust92
Copy link
Collaborator

Hey,

PRs are usually prioritized by importance (bugs and such) and community interest.

Keep in mind there are 600+ PRs to go over.

The provider team publishes a Roadmap every few months for topic they will focus on.

Also See: FAQ.

Hope this eases the uncertainty.

@yardensachs yardensachs force-pushed the f-timestreamwrite-database-resource branch 3 times, most recently from 8418998 to d6e67b6 Compare October 19, 2020 18:06
@yardensachs
Copy link
Contributor Author

@DrFaust92 Thanks for explaining.

So does this mean that this PR potentially can wait months to be merged?

@yardensachs yardensachs force-pushed the f-timestreamwrite-database-resource branch from d6e67b6 to 3737723 Compare November 4, 2020 23:02
@yardensachs yardensachs requested a review from a team as a code owner November 4, 2020 23:02
Base automatically changed from master to main January 23, 2021 00:59
@anGie44 anGie44 force-pushed the f-timestreamwrite-database-resource branch from 3737723 to a870d28 Compare May 7, 2021 16:46
@anGie44 anGie44 force-pushed the f-timestreamwrite-database-resource branch 4 times, most recently from 45fb494 to 5aa9e3e Compare May 8, 2021 01:04
@anGie44 anGie44 self-assigned this May 8, 2021
@anGie44
Copy link
Contributor

anGie44 commented May 8, 2021

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 <operation>WithTimeout and new testing conventions + new changelog process in the repo. I aim to get this is into the next release 👍

Output of acceptance tests (commercial):

--- PASS: TestAccAWSTimestreamWriteDatabase_basic (18.50s)
--- PASS: TestAccAWSTimestreamWriteDatabase_kmsKey (20.98s)
--- PASS: TestAccAWSTimestreamWriteDatabase_Tags (41.22s)
--- PASS: TestAccAWSTimestreamWriteDatabase_updateKmsKey (43.54s)

Output of acceptance tests (GovCloud):

--- SKIP: TestAccAWSTimestreamWriteDatabase_updateKmsKey (21.31s)
--- SKIP: TestAccAWSTimestreamWriteDatabase_Tags (23.66s)
--- SKIP: TestAccAWSTimestreamWriteDatabase_kmsKey (24.28s)
--- SKIP: TestAccAWSTimestreamWriteDatabase_basic (24.43s)

@anGie44 anGie44 force-pushed the f-timestreamwrite-database-resource branch from 5aa9e3e to 27fa37d Compare May 8, 2021 01:23
@anGie44 anGie44 added the new-resource Introduces a new resource. label May 8, 2021
@anGie44 anGie44 force-pushed the f-timestreamwrite-database-resource branch from 27fa37d to 13f913d Compare May 8, 2021 01:46
@anGie44 anGie44 added this to the v3.40.0 milestone May 8, 2021
@anGie44 anGie44 merged commit 3a36f47 into hashicorp:main May 8, 2021
@yardensachs yardensachs deleted the f-timestreamwrite-database-resource branch May 10, 2021 14:08
@ghost
Copy link

ghost commented May 13, 2021

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!

@github-actions
Copy link

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.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Introduces or discusses updates to documentation. new-resource Introduces a new resource. provider Pertains to the provider itself, rather than any interaction with AWS. service/timestreamwrite Issues and PRs that pertain to the timestreamwrite service. size/XL Managed by automation to categorize the size of a PR. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Resource: aws_timestreamwrite_database
4 participants