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

break out s3 endpoint service into tag based calls for interface and … #2

Closed
wants to merge 1 commit into from

Conversation

dennis-johnson-dev
Copy link

…gateway endpoints

Currently, when querying for a list of services to create VPC endpoints for, due to a recent change in AWS, we now get

"com.amazonaws.us-east-1.s3",
"com.amazonaws.us-east-1.s3",

back in the service name list which causes the data "aws_vpc_endpoint_service" data source to fail on finding duplicate items. The duplicate items comes from AWS creating endpoints for the s3 service for both interface and gateway types.

This approach requires manually tagging the VPC endpoints to work properly.

The consumers of this module don't require any updates to their interface or the list of services they are passing in. This module will break off the s3 service and handle it separately.

The long term approach is upgrading the AWS terraform provider to > 3.1 and utilizing the service_type terraform filter

@elementalvoid elementalvoid deleted the update_s3_vpc_endpoint branch February 16, 2022 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant