-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
r/aws_globalaccelerator_endpoint_group: Support port overrides #16121
r/aws_globalaccelerator_endpoint_group: Support port overrides #16121
Conversation
…, plus validations.
…ce tests. Acceptance test output: $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_basic' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAwsGlobalAcceleratorEndpointGroup_basic -timeout 120m === RUN TestAccAwsGlobalAcceleratorEndpointGroup_basic === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_basic === CONT TestAccAwsGlobalAcceleratorEndpointGroup_basic --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_basic (208.29s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 208.352s
Acceptance test output: $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_' ACCTEST_PARALLELISM=2 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 2 -run=TestAccAwsGlobalAcceleratorEndpointGroup_ -timeout 120m === RUN TestAccAwsGlobalAcceleratorEndpointGroup_basic === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_basic === RUN TestAccAwsGlobalAcceleratorEndpointGroup_disappears === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_disappears === RUN TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP === RUN TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint === RUN TestAccAwsGlobalAcceleratorEndpointGroup_update === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_update === RUN TestAccAwsGlobalAcceleratorEndpointGroup_tcp === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_tcp === CONT TestAccAwsGlobalAcceleratorEndpointGroup_basic === CONT TestAccAwsGlobalAcceleratorEndpointGroup_update --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_basic (220.05s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_tcp === CONT TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_update (270.33s) --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_tcp (240.05s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint (920.21s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_disappears --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP (1120.46s) --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_disappears (435.74s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 1816.104s
Acceptance test output: $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_' ACCTEST_PARALLELISM=2 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 2 -run=TestAccAwsGlobalAcceleratorEndpointGroup_ -timeout 120m === RUN TestAccAwsGlobalAcceleratorEndpointGroup_basic === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_basic === RUN TestAccAwsGlobalAcceleratorEndpointGroup_disappears === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_disappears === RUN TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP === RUN TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint === RUN TestAccAwsGlobalAcceleratorEndpointGroup_update === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_update === RUN TestAccAwsGlobalAcceleratorEndpointGroup_tcp === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_tcp === CONT TestAccAwsGlobalAcceleratorEndpointGroup_basic === CONT TestAccAwsGlobalAcceleratorEndpointGroup_update --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_basic (208.39s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_tcp --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_update (249.83s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_tcp (239.95s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP (698.28s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_disappears --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint (593.94s) --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_disappears (217.15s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 1165.315s
…torEndpointGroup_MultiRegion'. Acceptance test output: $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion' ACCTEST_PARALLELISM=2 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 2 -run=TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion -timeout 120m === RUN TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion === CONT TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion (211.73s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 211.776s
…d flatten. Acceptance test output: $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_' ACCTEST_PARALLELISM=2 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 2 -run=TestAccAwsGlobalAcceleratorEndpointGroup_ -timeout 120m === RUN TestAccAwsGlobalAcceleratorEndpointGroup_basic === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_basic === RUN TestAccAwsGlobalAcceleratorEndpointGroup_disappears === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_disappears === RUN TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP === RUN TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint === RUN TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion === RUN TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol === RUN TestAccAwsGlobalAcceleratorEndpointGroup_Update === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_Update === CONT TestAccAwsGlobalAcceleratorEndpointGroup_basic === CONT TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_basic (250.76s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_Update --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion (264.24s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol (219.10s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_Update (258.61s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint (645.67s) === CONT TestAccAwsGlobalAcceleratorEndpointGroup_disappears --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP (708.54s) --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_disappears (215.56s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 1370.737s
Acceptance test output: $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides' ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides -timeout 120m === RUN TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides === PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides === CONT TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides --- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides (279.43s) PASS ok github.com/terraform-providers/terraform-provider-aws/aws 279.479s
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
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_basic (229.93s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides (283.50s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol (280.32s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion (269.37s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint (647.61s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_disappears (251.07s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP (935.86s)
LGTM 🚀 Thanks @ewbankkit for the contribution! Verified Acceptance Tests in Commercial (us-west-2) make testacc TEST=./aws/ TESTARGS='-run=TestAccAwsGlobalAcceleratorEndpointGroup_' ACCTEST_PARALLELISM=2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -count 1 -parallel 2 -run=TestAccAwsGlobalAcceleratorEndpointGroup_ -timeout 120m
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_basic
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_basic
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_disappears
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_disappears
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol
=== RUN TestAccAwsGlobalAcceleratorEndpointGroup_Update
=== PAUSE TestAccAwsGlobalAcceleratorEndpointGroup_Update
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_basic
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_basic (207.99s)
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_Update
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_PortOverrides (265.52s)
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_Update (306.03s)
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_TCPHealthCheckProtocol (282.39s)
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_MultiRegion (232.22s)
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_InstanceEndpoint (676.65s)
=== CONT TestAccAwsGlobalAcceleratorEndpointGroup_disappears
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_disappears (359.03s)
--- PASS: TestAccAwsGlobalAcceleratorEndpointGroup_ALBEndpoint_ClientIP (851.48s)
PASS
ok github.com/terraform-providers/terraform-provider-aws/aws 1633.172s |
This has been released in version 3.16.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 for triage. Thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Closes: #15787.
Release note for CHANGELOG:
Output from acceptance testing: