-
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
Amazon ECS Service Connect #28052
Amazon ECS Service Connect #28052
Conversation
Community NoteVoting for Prioritization
For Submitters
|
This comment was marked as outdated.
This comment was marked as outdated.
This reverts commit ebcbabc.
This reverts commit e5495cc.
This reverts commit 6a2d9b7.
…nnot be set with Required'.
Acceptance test output: % make testacc TESTARGS='-run=TestAccECSCluster_\|TestAccECSClusterDataSource_' PKG=ecs ACCTEST_PARALLELISM=3 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/ecs/... -v -count 1 -parallel 3 -run=TestAccECSCluster_\|TestAccECSClusterDataSource_ -timeout 180m === RUN TestAccECSClusterDataSource_ecsCluster === PAUSE TestAccECSClusterDataSource_ecsCluster === RUN TestAccECSClusterDataSource_ecsClusterContainerInsights === PAUSE TestAccECSClusterDataSource_ecsClusterContainerInsights === RUN TestAccECSCluster_basic === PAUSE TestAccECSCluster_basic === RUN TestAccECSCluster_disappears === PAUSE TestAccECSCluster_disappears === RUN TestAccECSCluster_tags === PAUSE TestAccECSCluster_tags === RUN TestAccECSCluster_serviceConnectDefaults === PAUSE TestAccECSCluster_serviceConnectDefaults === RUN TestAccECSCluster_singleCapacityProvider === PAUSE TestAccECSCluster_singleCapacityProvider === RUN TestAccECSCluster_capacityProviders === PAUSE TestAccECSCluster_capacityProviders === RUN TestAccECSCluster_capacityProvidersUpdate === PAUSE TestAccECSCluster_capacityProvidersUpdate === RUN TestAccECSCluster_capacityProvidersNoStrategy === PAUSE TestAccECSCluster_capacityProvidersNoStrategy === RUN TestAccECSCluster_containerInsights === PAUSE TestAccECSCluster_containerInsights === RUN TestAccECSCluster_configuration === PAUSE TestAccECSCluster_configuration === CONT TestAccECSClusterDataSource_ecsCluster === CONT TestAccECSCluster_configuration === CONT TestAccECSCluster_containerInsights --- PASS: TestAccECSClusterDataSource_ecsCluster (31.39s) === CONT TestAccECSCluster_capacityProvidersNoStrategy --- PASS: TestAccECSCluster_configuration (63.01s) === CONT TestAccECSCluster_capacityProvidersUpdate --- PASS: TestAccECSCluster_containerInsights (81.57s) === CONT TestAccECSCluster_capacityProviders --- PASS: TestAccECSCluster_capacityProvidersNoStrategy (58.52s) === CONT TestAccECSCluster_singleCapacityProvider --- PASS: TestAccECSCluster_capacityProviders (42.00s) === CONT TestAccECSCluster_serviceConnectDefaults --- PASS: TestAccECSCluster_capacityProvidersUpdate (82.57s) === CONT TestAccECSCluster_tags --- PASS: TestAccECSCluster_singleCapacityProvider (78.18s) === CONT TestAccECSCluster_disappears --- PASS: TestAccECSCluster_serviceConnectDefaults (142.47s) === CONT TestAccECSCluster_basic --- PASS: TestAccECSCluster_disappears (24.96s) === CONT TestAccECSClusterDataSource_ecsClusterContainerInsights --- PASS: TestAccECSCluster_tags (55.91s) --- PASS: TestAccECSCluster_basic (29.97s) --- PASS: TestAccECSClusterDataSource_ecsClusterContainerInsights (28.05s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/ecs 229.778s
… errors (ECS Service Connect).
…h a single newline character'.
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=TestAccECSCluster_\|TestAccECSClusterDataSource_' PKG=ecs ACCTEST_PARALLELISM=3
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/ecs/... -v -count 1 -parallel 3 -run=TestAccECSCluster_\|TestAccECSClusterDataSource_ -timeout 180m
=== RUN TestAccECSClusterDataSource_ecsCluster
=== PAUSE TestAccECSClusterDataSource_ecsCluster
=== RUN TestAccECSClusterDataSource_ecsClusterContainerInsights
=== PAUSE TestAccECSClusterDataSource_ecsClusterContainerInsights
=== RUN TestAccECSCluster_basic
=== PAUSE TestAccECSCluster_basic
=== RUN TestAccECSCluster_disappears
=== PAUSE TestAccECSCluster_disappears
=== RUN TestAccECSCluster_tags
=== PAUSE TestAccECSCluster_tags
=== RUN TestAccECSCluster_serviceConnectDefaults
=== PAUSE TestAccECSCluster_serviceConnectDefaults
=== RUN TestAccECSCluster_singleCapacityProvider
=== PAUSE TestAccECSCluster_singleCapacityProvider
=== RUN TestAccECSCluster_capacityProviders
=== PAUSE TestAccECSCluster_capacityProviders
=== RUN TestAccECSCluster_capacityProvidersUpdate
=== PAUSE TestAccECSCluster_capacityProvidersUpdate
=== RUN TestAccECSCluster_capacityProvidersNoStrategy
=== PAUSE TestAccECSCluster_capacityProvidersNoStrategy
=== RUN TestAccECSCluster_containerInsights
=== PAUSE TestAccECSCluster_containerInsights
=== RUN TestAccECSCluster_configuration
=== PAUSE TestAccECSCluster_configuration
=== CONT TestAccECSClusterDataSource_ecsCluster
=== CONT TestAccECSCluster_configuration
=== CONT TestAccECSCluster_containerInsights
--- PASS: TestAccECSClusterDataSource_ecsCluster (31.39s)
=== CONT TestAccECSCluster_capacityProvidersNoStrategy
--- PASS: TestAccECSCluster_configuration (63.01s)
=== CONT TestAccECSCluster_capacityProvidersUpdate
--- PASS: TestAccECSCluster_containerInsights (81.57s)
=== CONT TestAccECSCluster_capacityProviders
--- PASS: TestAccECSCluster_capacityProvidersNoStrategy (58.52s)
=== CONT TestAccECSCluster_singleCapacityProvider
--- PASS: TestAccECSCluster_capacityProviders (42.00s)
=== CONT TestAccECSCluster_serviceConnectDefaults
--- PASS: TestAccECSCluster_capacityProvidersUpdate (82.57s)
=== CONT TestAccECSCluster_tags
--- PASS: TestAccECSCluster_singleCapacityProvider (78.18s)
=== CONT TestAccECSCluster_disappears
--- PASS: TestAccECSCluster_serviceConnectDefaults (142.47s)
=== CONT TestAccECSCluster_basic
--- PASS: TestAccECSCluster_disappears (24.96s)
=== CONT TestAccECSClusterDataSource_ecsClusterContainerInsights
--- PASS: TestAccECSCluster_tags (55.91s)
--- PASS: TestAccECSCluster_basic (29.97s)
--- PASS: TestAccECSClusterDataSource_ecsClusterContainerInsights (28.05s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/ecs 229.778s
% make testacc TESTARGS='-run=TestAccECSService_' PKG=ecs ACCTEST_PARALLELISM=3
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/ecs/... -v -count 1 -parallel 3 -run=TestAccECSService_ -timeout 180m
=== RUN TestAccECSService_basic
=== PAUSE TestAccECSService_basic
=== RUN TestAccECSService_basicImport
=== PAUSE TestAccECSService_basicImport
=== RUN TestAccECSService_disappears
=== PAUSE TestAccECSService_disappears
=== RUN TestAccECSService_PlacementStrategy_unnormalized
=== PAUSE TestAccECSService_PlacementStrategy_unnormalized
=== RUN TestAccECSService_CapacityProviderStrategy_basic
=== PAUSE TestAccECSService_CapacityProviderStrategy_basic
=== RUN TestAccECSService_CapacityProviderStrategy_forceNewDeployment
=== PAUSE TestAccECSService_CapacityProviderStrategy_forceNewDeployment
=== RUN TestAccECSService_CapacityProviderStrategy_update
=== PAUSE TestAccECSService_CapacityProviderStrategy_update
=== RUN TestAccECSService_CapacityProviderStrategy_multiple
=== PAUSE TestAccECSService_CapacityProviderStrategy_multiple
=== RUN TestAccECSService_familyAndRevision
=== PAUSE TestAccECSService_familyAndRevision
=== RUN TestAccECSService_renamedCluster
=== PAUSE TestAccECSService_renamedCluster
=== RUN TestAccECSService_healthCheckGracePeriodSeconds
=== PAUSE TestAccECSService_healthCheckGracePeriodSeconds
=== RUN TestAccECSService_iamRole
=== PAUSE TestAccECSService_iamRole
=== RUN TestAccECSService_DeploymentControllerType_codeDeploy
=== PAUSE TestAccECSService_DeploymentControllerType_codeDeploy
=== RUN TestAccECSService_DeploymentControllerType_codeDeployUpdateDesiredCountAndHealthCheckGracePeriod
=== PAUSE TestAccECSService_DeploymentControllerType_codeDeployUpdateDesiredCountAndHealthCheckGracePeriod
=== RUN TestAccECSService_DeploymentControllerType_external
=== PAUSE TestAccECSService_DeploymentControllerType_external
=== RUN TestAccECSService_DeploymentValues_basic
=== PAUSE TestAccECSService_DeploymentValues_basic
=== RUN TestAccECSService_DeploymentValues_minZeroMaxOneHundred
=== PAUSE TestAccECSService_DeploymentValues_minZeroMaxOneHundred
=== RUN TestAccECSService_deploymentCircuitBreaker
=== PAUSE TestAccECSService_deploymentCircuitBreaker
=== RUN TestAccECSService_loadBalancerChanges
=== PAUSE TestAccECSService_loadBalancerChanges
=== RUN TestAccECSService_clusterName
=== PAUSE TestAccECSService_clusterName
=== RUN TestAccECSService_alb
=== PAUSE TestAccECSService_alb
=== RUN TestAccECSService_multipleTargetGroups
=== PAUSE TestAccECSService_multipleTargetGroups
=== RUN TestAccECSService_forceNewDeployment
=== PAUSE TestAccECSService_forceNewDeployment
=== RUN TestAccECSService_forceNewDeploymentTriggers
=== PAUSE TestAccECSService_forceNewDeploymentTriggers
=== RUN TestAccECSService_PlacementStrategy_basic
=== PAUSE TestAccECSService_PlacementStrategy_basic
=== RUN TestAccECSService_PlacementStrategy_missing
=== PAUSE TestAccECSService_PlacementStrategy_missing
=== RUN TestAccECSService_PlacementConstraints_basic
=== PAUSE TestAccECSService_PlacementConstraints_basic
=== RUN TestAccECSService_PlacementConstraints_emptyExpression
=== PAUSE TestAccECSService_PlacementConstraints_emptyExpression
=== RUN TestAccECSService_LaunchTypeFargate_basic
=== PAUSE TestAccECSService_LaunchTypeFargate_basic
=== RUN TestAccECSService_LaunchTypeFargate_platformVersion
=== PAUSE TestAccECSService_LaunchTypeFargate_platformVersion
=== RUN TestAccECSService_LaunchTypeFargate_waitForSteadyState
=== PAUSE TestAccECSService_LaunchTypeFargate_waitForSteadyState
=== RUN TestAccECSService_LaunchTypeFargate_updateWaitForSteadyState
=== PAUSE TestAccECSService_LaunchTypeFargate_updateWaitForSteadyState
=== RUN TestAccECSService_LaunchTypeEC2_network
=== PAUSE TestAccECSService_LaunchTypeEC2_network
=== RUN TestAccECSService_DaemonSchedulingStrategy_basic
=== PAUSE TestAccECSService_DaemonSchedulingStrategy_basic
=== RUN TestAccECSService_DaemonSchedulingStrategy_setDeploymentMinimum
=== PAUSE TestAccECSService_DaemonSchedulingStrategy_setDeploymentMinimum
=== RUN TestAccECSService_replicaSchedulingStrategy
=== PAUSE TestAccECSService_replicaSchedulingStrategy
=== RUN TestAccECSService_ServiceRegistries_basic
=== PAUSE TestAccECSService_ServiceRegistries_basic
=== RUN TestAccECSService_ServiceRegistries_container
=== PAUSE TestAccECSService_ServiceRegistries_container
=== RUN TestAccECSService_ServiceRegistries_changes
=== PAUSE TestAccECSService_ServiceRegistries_changes
=== RUN TestAccECSService_ServiceConnect_basic
=== PAUSE TestAccECSService_ServiceConnect_basic
=== RUN TestAccECSService_ServiceConnect_full
=== PAUSE TestAccECSService_ServiceConnect_full
=== RUN TestAccECSService_Tags_basic
=== PAUSE TestAccECSService_Tags_basic
=== RUN TestAccECSService_Tags_managed
=== PAUSE TestAccECSService_Tags_managed
=== RUN TestAccECSService_Tags_propagate
=== PAUSE TestAccECSService_Tags_propagate
=== RUN TestAccECSService_executeCommand
=== PAUSE TestAccECSService_executeCommand
=== CONT TestAccECSService_basic
=== CONT TestAccECSService_forceNewDeploymentTriggers
=== CONT TestAccECSService_DeploymentControllerType_codeDeploy
--- PASS: TestAccECSService_forceNewDeploymentTriggers (81.36s)
=== CONT TestAccECSService_forceNewDeployment
--- PASS: TestAccECSService_basic (91.51s)
=== CONT TestAccECSService_multipleTargetGroups
--- PASS: TestAccECSService_forceNewDeployment (88.20s)
=== CONT TestAccECSService_alb
--- PASS: TestAccECSService_DeploymentControllerType_codeDeploy (222.72s)
=== CONT TestAccECSService_clusterName
--- PASS: TestAccECSService_clusterName (84.66s)
=== CONT TestAccECSService_loadBalancerChanges
--- PASS: TestAccECSService_multipleTargetGroups (227.17s)
=== CONT TestAccECSService_deploymentCircuitBreaker
--- PASS: TestAccECSService_alb (222.31s)
=== CONT TestAccECSService_DeploymentValues_minZeroMaxOneHundred
--- PASS: TestAccECSService_deploymentCircuitBreaker (84.78s)
=== CONT TestAccECSService_DeploymentValues_basic
--- PASS: TestAccECSService_DeploymentValues_minZeroMaxOneHundred (85.54s)
=== CONT TestAccECSService_DeploymentControllerType_external
--- PASS: TestAccECSService_DeploymentValues_basic (85.04s)
=== CONT TestAccECSService_DeploymentControllerType_codeDeployUpdateDesiredCountAndHealthCheckGracePeriod
--- PASS: TestAccECSService_loadBalancerChanges (182.68s)
=== CONT TestAccECSService_DaemonSchedulingStrategy_setDeploymentMinimum
--- PASS: TestAccECSService_DaemonSchedulingStrategy_setDeploymentMinimum (28.82s)
=== CONT TestAccECSService_executeCommand
--- PASS: TestAccECSService_DeploymentControllerType_external (45.49s)
=== CONT TestAccECSService_Tags_propagate
--- PASS: TestAccECSService_executeCommand (78.81s)
=== CONT TestAccECSService_Tags_managed
--- PASS: TestAccECSService_Tags_propagate (99.74s)
=== CONT TestAccECSService_Tags_basic
--- PASS: TestAccECSService_Tags_managed (75.59s)
=== CONT TestAccECSService_ServiceConnect_full
--- PASS: TestAccECSService_Tags_basic (105.85s)
=== CONT TestAccECSService_ServiceConnect_basic
--- PASS: TestAccECSService_ServiceConnect_full (164.27s)
=== CONT TestAccECSService_ServiceRegistries_changes
--- PASS: TestAccECSService_DeploymentControllerType_codeDeployUpdateDesiredCountAndHealthCheckGracePeriod (422.34s)
=== CONT TestAccECSService_ServiceRegistries_container
--- PASS: TestAccECSService_ServiceConnect_basic (184.95s)
=== CONT TestAccECSService_ServiceRegistries_basic
--- PASS: TestAccECSService_ServiceRegistries_basic (156.65s)
=== CONT TestAccECSService_replicaSchedulingStrategy
--- PASS: TestAccECSService_ServiceRegistries_container (167.81s)
=== CONT TestAccECSService_PlacementConstraints_basic
--- PASS: TestAccECSService_ServiceRegistries_changes (253.36s)
=== CONT TestAccECSService_LaunchTypeFargate_basic
--- PASS: TestAccECSService_replicaSchedulingStrategy (95.08s)
=== CONT TestAccECSService_PlacementConstraints_emptyExpression
--- PASS: TestAccECSService_PlacementConstraints_basic (87.14s)
=== CONT TestAccECSService_LaunchTypeFargate_platformVersion
--- PASS: TestAccECSService_LaunchTypeFargate_basic (141.72s)
=== CONT TestAccECSService_LaunchTypeFargate_updateWaitForSteadyState
--- PASS: TestAccECSService_PlacementConstraints_emptyExpression (74.63s)
=== CONT TestAccECSService_LaunchTypeFargate_waitForSteadyState
--- PASS: TestAccECSService_LaunchTypeFargate_platformVersion (151.51s)
=== CONT TestAccECSService_LaunchTypeEC2_network
--- PASS: TestAccECSService_LaunchTypeFargate_waitForSteadyState (158.12s)
=== CONT TestAccECSService_DaemonSchedulingStrategy_basic
--- PASS: TestAccECSService_LaunchTypeFargate_updateWaitForSteadyState (168.55s)
=== CONT TestAccECSService_CapacityProviderStrategy_update
--- PASS: TestAccECSService_LaunchTypeEC2_network (84.80s)
=== CONT TestAccECSService_iamRole
--- PASS: TestAccECSService_DaemonSchedulingStrategy_basic (31.36s)
=== CONT TestAccECSService_healthCheckGracePeriodSeconds
--- PASS: TestAccECSService_iamRole (63.74s)
=== CONT TestAccECSService_renamedCluster
--- PASS: TestAccECSService_renamedCluster (87.07s)
=== CONT TestAccECSService_familyAndRevision
--- PASS: TestAccECSService_familyAndRevision (85.83s)
=== CONT TestAccECSService_CapacityProviderStrategy_multiple
--- PASS: TestAccECSService_healthCheckGracePeriodSeconds (303.66s)
=== CONT TestAccECSService_PlacementStrategy_missing
--- PASS: TestAccECSService_PlacementStrategy_missing (1.15s)
=== CONT TestAccECSService_PlacementStrategy_basic
--- PASS: TestAccECSService_CapacityProviderStrategy_update (334.64s)
=== CONT TestAccECSService_disappears
--- PASS: TestAccECSService_PlacementStrategy_basic (114.46s)
=== CONT TestAccECSService_basicImport
--- PASS: TestAccECSService_disappears (193.21s)
=== CONT TestAccECSService_CapacityProviderStrategy_forceNewDeployment
--- PASS: TestAccECSService_basicImport (201.09s)
=== CONT TestAccECSService_CapacityProviderStrategy_basic
--- PASS: TestAccECSService_CapacityProviderStrategy_forceNewDeployment (158.62s)
=== CONT TestAccECSService_PlacementStrategy_unnormalized
--- PASS: TestAccECSService_PlacementStrategy_unnormalized (76.52s)
--- PASS: TestAccECSService_CapacityProviderStrategy_basic (221.43s)
--- PASS: TestAccECSService_CapacityProviderStrategy_multiple (1012.45s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/ecs 2656.000s
@meetreks @johnsonaj Thanks for the contribution 🎉 👏. |
This functionality has been released in v4.43.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
The PR is to cater for newly released ECS feature service connect.
Relations
Closes #28043
References
Output from Acceptance Testing