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

v2.69.1 proposal, ensure AWS terraform provider v2.70.0 compatibility #587

Closed
wants to merge 2 commits into from
Closed

v2.69.1 proposal, ensure AWS terraform provider v2.70.0 compatibility #587

wants to merge 2 commits into from

Conversation

circa10a
Copy link

@circa10a circa10a commented Feb 6, 2021

Description

  • Remove s3/dynamodb service endpoint data sources due to incompatible service_type parameter.
  • Replace dynamic service name from data sources with template strings

Motivation and Context

After this week's release of privatelink for s3, the fixes implemented in v2.69.0 and v2.70.0 don't mitigate errors when using the AWS Terraform provider v2.X.X

Addresses:

Breaking Changes

This is a proposed patch release for v2.69. which did not resolve the original issue.

How Has This Been Tested?

This was tested by creating vpc endpoint services with AWS Terraform Provider 2.70.0 and that no error occurred.

/cc @tbugfinder

@circa10a
Copy link
Author

Any thoughts on this? If none, I'll go ahead and close it.

@antonbabenko
Copy link
Member

Hi @circa10a ! This looks pretty good, I will take a deeper look at this in a few days and verify how it works.

There is no need to close this one.

@bryantbiggs
Copy link
Member

hi @circa10a - thanks for the PR! this is a great idea and I have taken this further in #597. a couple things to note:

  • DynamoDB interface endpoints are currently not supported yet so I have not included that change as you have
  • If users were to try to use the S3 interface type, it would fail. I have addressed this as well
  • I've updated an example project as well - this is what we use to validate as well as demonstrate changes

@circa10a
Copy link
Author

Closing in favor of #597

@circa10a circa10a closed this Feb 22, 2021
@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 Oct 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants