-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Enhancement] Add resource_configuration_arn
and service_network_arn
to vpc_endpoint
#41116
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
[Enhancement] Add resource_configuration_arn
and service_network_arn
to vpc_endpoint
#41116
Conversation
Community NoteVoting for Prioritization
For Submitters
|
…re getting mixed case values, this gets Resource type endpoints working
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.
LGTM 🚀.
% make testacc TESTARGS='-run=TestAccVPCEndpoint_\|TestAccVPCEndpointPolicy_\|TestAccVPCEndpointSubnetAssociation_' PKG=ec2 ACCTEST_PARALLELISM=4
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.3 test ./internal/service/ec2/... -v -count 1 -parallel 4 -run=TestAccVPCEndpoint_\|TestAccVPCEndpointPolicy_\|TestAccVPCEndpointSubnetAssociation_ -timeout 360m -vet=off
2025/01/28 17:48:03 Initializing Terraform AWS Provider...
=== RUN TestAccVPCEndpointPolicy_basic
=== PAUSE TestAccVPCEndpointPolicy_basic
=== RUN TestAccVPCEndpointPolicy_disappears
=== PAUSE TestAccVPCEndpointPolicy_disappears
=== RUN TestAccVPCEndpointPolicy_disappears_endpoint
=== PAUSE TestAccVPCEndpointPolicy_disappears_endpoint
=== RUN TestAccVPCEndpointSubnetAssociation_basic
=== PAUSE TestAccVPCEndpointSubnetAssociation_basic
=== RUN TestAccVPCEndpointSubnetAssociation_disappears
=== PAUSE TestAccVPCEndpointSubnetAssociation_disappears
=== RUN TestAccVPCEndpointSubnetAssociation_multiple
=== PAUSE TestAccVPCEndpointSubnetAssociation_multiple
=== RUN TestAccVPCEndpoint_gatewayBasic
=== PAUSE TestAccVPCEndpoint_gatewayBasic
=== RUN TestAccVPCEndpoint_interfaceBasic
=== PAUSE TestAccVPCEndpoint_interfaceBasic
=== RUN TestAccVPCEndpoint_interfaceNoPrivateDNS
=== PAUSE TestAccVPCEndpoint_interfaceNoPrivateDNS
=== RUN TestAccVPCEndpoint_interfacePrivateDNS
=== PAUSE TestAccVPCEndpoint_interfacePrivateDNS
=== RUN TestAccVPCEndpoint_interfacePrivateDNSNoGateway
=== PAUSE TestAccVPCEndpoint_interfacePrivateDNSNoGateway
=== RUN TestAccVPCEndpoint_disappears
=== PAUSE TestAccVPCEndpoint_disappears
=== RUN TestAccVPCEndpoint_tags
=== PAUSE TestAccVPCEndpoint_tags
=== RUN TestAccVPCEndpoint_gatewayWithRouteTableAndPolicy
=== PAUSE TestAccVPCEndpoint_gatewayWithRouteTableAndPolicy
=== RUN TestAccVPCEndpoint_gatewayPolicy
=== PAUSE TestAccVPCEndpoint_gatewayPolicy
=== RUN TestAccVPCEndpoint_ignoreEquivalent
=== PAUSE TestAccVPCEndpoint_ignoreEquivalent
=== RUN TestAccVPCEndpoint_ipAddressType
=== PAUSE TestAccVPCEndpoint_ipAddressType
=== RUN TestAccVPCEndpoint_interfaceWithSubnetAndSecurityGroup
=== PAUSE TestAccVPCEndpoint_interfaceWithSubnetAndSecurityGroup
=== RUN TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnCreate
=== PAUSE TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnCreate
=== RUN TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnUpdate
=== PAUSE TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnUpdate
=== RUN TestAccVPCEndpoint_interfaceUserDefinedIPv4
=== PAUSE TestAccVPCEndpoint_interfaceUserDefinedIPv4
=== RUN TestAccVPCEndpoint_interfaceUserDefinedIPv6
=== PAUSE TestAccVPCEndpoint_interfaceUserDefinedIPv6
=== RUN TestAccVPCEndpoint_interfaceUserDefinedDualstack
=== PAUSE TestAccVPCEndpoint_interfaceUserDefinedDualstack
=== RUN TestAccVPCEndpoint_VPCEndpointType_gatewayLoadBalancer
=== PAUSE TestAccVPCEndpoint_VPCEndpointType_gatewayLoadBalancer
=== RUN TestAccVPCEndpoint_crossRegionService
=== PAUSE TestAccVPCEndpoint_crossRegionService
=== RUN TestAccVPCEndpoint_invalidCrossRegionService
=== PAUSE TestAccVPCEndpoint_invalidCrossRegionService
=== RUN TestAccVPCEndpoint_resourceConfiguration
=== PAUSE TestAccVPCEndpoint_resourceConfiguration
=== RUN TestAccVPCEndpoint_serviceNetwork
=== PAUSE TestAccVPCEndpoint_serviceNetwork
=== CONT TestAccVPCEndpointPolicy_basic
=== CONT TestAccVPCEndpoint_gatewayPolicy
=== CONT TestAccVPCEndpoint_interfaceBasic
=== CONT TestAccVPCEndpoint_interfacePrivateDNSNoGateway
--- PASS: TestAccVPCEndpoint_gatewayPolicy (48.08s)
=== CONT TestAccVPCEndpoint_disappears
--- PASS: TestAccVPCEndpointPolicy_basic (58.67s)
=== CONT TestAccVPCEndpoint_interfacePrivateDNS
--- PASS: TestAccVPCEndpoint_disappears (26.68s)
=== CONT TestAccVPCEndpoint_interfaceNoPrivateDNS
--- PASS: TestAccVPCEndpoint_interfaceBasic (80.02s)
=== CONT TestAccVPCEndpoint_gatewayWithRouteTableAndPolicy
--- PASS: TestAccVPCEndpoint_gatewayWithRouteTableAndPolicy (46.54s)
=== CONT TestAccVPCEndpoint_tags
--- PASS: TestAccVPCEndpoint_interfaceNoPrivateDNS (74.14s)
=== CONT TestAccVPCEndpoint_interfaceUserDefinedIPv6
--- PASS: TestAccVPCEndpoint_tags (52.20s)
=== CONT TestAccVPCEndpoint_serviceNetwork
--- PASS: TestAccVPCEndpoint_serviceNetwork (64.16s)
=== CONT TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnCreate
--- PASS: TestAccVPCEndpoint_interfaceUserDefinedIPv6 (194.14s)
=== CONT TestAccVPCEndpoint_resourceConfiguration
--- PASS: TestAccVPCEndpoint_interfacePrivateDNSNoGateway (446.26s)
=== CONT TestAccVPCEndpoint_invalidCrossRegionService
--- PASS: TestAccVPCEndpoint_resourceConfiguration (114.99s)
=== CONT TestAccVPCEndpoint_interfaceUserDefinedIPv4
--- PASS: TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnCreate (258.75s)
=== CONT TestAccVPCEndpoint_crossRegionService
=== CONT TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnUpdate
--- PASS: TestAccVPCEndpoint_interfacePrivateDNS (532.64s)
--- PASS: TestAccVPCEndpoint_interfaceUserDefinedIPv4 (217.26s)
=== CONT TestAccVPCEndpoint_VPCEndpointType_gatewayLoadBalancer
--- PASS: TestAccVPCEndpoint_invalidCrossRegionService (231.59s)
=== CONT TestAccVPCEndpointSubnetAssociation_disappears
--- PASS: TestAccVPCEndpoint_interfaceNonAWSServiceAcceptOnUpdate (274.33s)
=== CONT TestAccVPCEndpoint_interfaceUserDefinedDualstack
--- PASS: TestAccVPCEndpointSubnetAssociation_disappears (228.03s)
=== CONT TestAccVPCEndpoint_gatewayBasic
--- PASS: TestAccVPCEndpoint_gatewayBasic (28.62s)
=== CONT TestAccVPCEndpoint_ipAddressType
--- PASS: TestAccVPCEndpoint_crossRegionService (478.83s)
=== CONT TestAccVPCEndpointSubnetAssociation_multiple
--- PASS: TestAccVPCEndpoint_interfaceUserDefinedDualstack (193.11s)
=== CONT TestAccVPCEndpoint_interfaceWithSubnetAndSecurityGroup
--- PASS: TestAccVPCEndpoint_VPCEndpointType_gatewayLoadBalancer (391.54s)
=== CONT TestAccVPCEndpointPolicy_disappears
--- PASS: TestAccVPCEndpointPolicy_disappears (44.45s)
=== CONT TestAccVPCEndpointPolicy_disappears_endpoint
--- PASS: TestAccVPCEndpointPolicy_disappears_endpoint (33.24s)
=== CONT TestAccVPCEndpointSubnetAssociation_basic
--- PASS: TestAccVPCEndpoint_ipAddressType (286.87s)
=== CONT TestAccVPCEndpoint_ignoreEquivalent
--- PASS: TestAccVPCEndpoint_ignoreEquivalent (32.09s)
--- PASS: TestAccVPCEndpoint_interfaceWithSubnetAndSecurityGroup (244.29s)
--- PASS: TestAccVPCEndpointSubnetAssociation_basic (204.84s)
--- PASS: TestAccVPCEndpointSubnetAssociation_multiple (390.84s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/ec2 1377.090s
% make testacc TESTARGS='-run=TestAccVPCEndpoint_gatewayBasic\|TestAccVPCEndpoint_interfaceBasic' PKG=ec2 make: Verifying source code with gofmt... ==> Checking that code complies with gofmt requirements... TF_ACC=1 go1.23.3 test ./internal/service/ec2/... -v -count 1 -parallel 20 -run=TestAccVPCEndpoint_gatewayBasic\|TestAccVPCEndpoint_interfaceBasic -timeout 360m -vet=off 2025/01/29 07:44:31 Initializing Terraform AWS Provider... === RUN TestAccVPCEndpoint_gatewayBasic === PAUSE TestAccVPCEndpoint_gatewayBasic === RUN TestAccVPCEndpoint_interfaceBasic === PAUSE TestAccVPCEndpoint_interfaceBasic === CONT TestAccVPCEndpoint_gatewayBasic === CONT TestAccVPCEndpoint_interfaceBasic --- PASS: TestAccVPCEndpoint_gatewayBasic (34.97s) --- PASS: TestAccVPCEndpoint_interfaceBasic (54.68s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/ec2 60.448s
% make testacc-tflint
Installing "aws" plugin...
Installed "aws" (source: github.com/terraform-linters/tflint-ruleset-aws, version: 0.37.0)
make: Acceptance Test Linting (standalone) / tflint...
make: Acceptance Test Linting (embedded) / tflint... |
…nvalid_certificate' has been removed.
@drewtul Thanks for the contribution 🎉 👏. |
This functionality has been released in v5.85.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. Thank you! |
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. |
Description
Add
resource_configuration_arn
andservice_network_arn
tovpc_endpoint
Note there's some case differences in the values of
State
returned from the API depending on the type of the VPC Endpoint. This PR normalises the comparisons of state to lowercase against the constants inconsts.go
and does not use theState
enum from the SDK.Relations
Relates #40384
References
Output from Acceptance Testing