[WIP] F amazon bedrock

[skyscrapr]

Description

Amazon Bedrock New Service Implementation

Relations

Closes #33809.
Closes #33738.
Closes #33737.
Closes #33718.

References

Amazon Bedrock API

Output from Acceptance Testing

TBD

% make testacc TESTS=TestAccXXX PKG=ec2

...

[skyscrapr]

There are a number of areas that need design discussion:

resource_aws_bedrock_custom_model:
-- I chose to not export the model customisation job as a resource since this cannot be managed or deleted.
-- Instead I chose to create a custom model resource and wrap the customisation job in that handling.
-- This has a couple of implications for tag handling.
-- What is the best way to handle failures for resources with long creation times? I've chosen to save the ID's in state as early as possible to facilitate destroy/create by the built-in taint behaviour. Is that a good approach?
-- I'll upload the test output if the current implementation is approved.

@wellsiau-aws feel free to loop in anyone that might be interested in reviewing.

[breathingdust]

Hi @skyscrapr! 👋 Thanks for the contribution, I know it’s WIP but I noticed a few things at first glance

This week we are finally making a policy change to require that net new resources are based on AWS Go SDK v2 and terraform-plugin-framework rather than terraform-plugin-sdk. These resources will need to be updated to confirm as a result. It should not be complex to do, but do reach out if you have any questions.

Additionally, if possible we also like to see one PR per resource/data source when ready for final review. It can be very difficult/inefficient for maintainers to review PR’s with multiple additions.

Thank you!

[skyscrapr]

I'm opening a bunch of new ones PRs to cover the recommendation above.
Please close.

[AdamTylerLynch]

I'm opening a bunch of new ones PRs to cover the recommendation above. Please close.

@skyscrapr do you want us to close this PR without any merge? Do you new PRs #34042 and #34043 supersede this PR?

[skyscrapr]

@AdamTylerLynch

This PR is against sdk v1.
New PRs are against sdk v2.

As soon as the current open PRs are approved, I'll raise the remaining PRs for the other data sources and resources.

This one can be closed 👍. Thanks

[skyscrapr]

I'm closing this as the new PRs have all been raised. Thanks!

[github-actions]

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.