-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Merged
ewbankkit
merged 15 commits into
hashicorp:main
from
drewtul:f-vpcendpoint-resource-configuration
Jan 29, 2025
Merged
[Enhancement] Add resource_configuration_arn
and service_network_arn
to vpc_endpoint
#41116
ewbankkit
merged 15 commits into
hashicorp:main
from
drewtul:f-vpcendpoint-resource-configuration
Jan 29, 2025
+311
−35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Community NoteVoting for Prioritization
For Submitters
|
github-actions
bot
added
documentation
Introduces or discusses updates to documentation.
service/vpc
Issues and PRs that pertain to the vpc service.
needs-triage
Waiting for first response or review from a maintainer.
partner
Contribution from a partner.
labels
Jan 28, 2025
ewbankkit
reviewed
Jan 28, 2025
justinretzolk
added
enhancement
Requests to existing resources that expand the functionality or scope.
and removed
needs-triage
Waiting for first response or review from a maintainer.
labels
Jan 28, 2025
…re getting mixed case values, this gets Resource type endpoints working
github-actions
bot
added
tests
PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
sweeper
Pertains to changes to or issues with the sweeper.
labels
Jan 28, 2025
github-actions
bot
added
the
prioritized
Part of the maintainer teams immediate focus. To be addressed within the current quarter.
label
Jan 28, 2025
ewbankkit
previously approved these changes
Jan 28, 2025
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.
ewbankkit
requested review from
breathingdust and
justinretzolk
as code owners
January 29, 2025 13:13
YakDriver
approved these changes
Jan 29, 2025
@drewtul Thanks for the contribution 🎉 👏. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
documentation
Introduces or discusses updates to documentation.
enhancement
Requests to existing resources that expand the functionality or scope.
partner
Contribution from a partner.
prioritized
Part of the maintainer teams immediate focus. To be addressed within the current quarter.
service/vpc
Issues and PRs that pertain to the vpc service.
sweeper
Pertains to changes to or issues with the sweeper.
tests
PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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