Skip to content

Commit fd05c10

Browse files
authored
chore(ecs-patterns): fix ec2/multiple-application-load-balanced-ecs-service integration test (#32524)
### Issue # (if applicable) Relate #32506 ### Reason for this change This test is failing. Please try this quick check ```bash docker run -d -p 80:80 -p 90:90 amazon/amazon-ecs-sample:latest # Success curl http://localhost:80 # Fail curl http://localhost:90 ``` ### Description of changes - Modify ECS task to also expose port 90 - Migrate to LaunchTemplate as LaunchConfiguration cannot be used on new account ### Description of how you validated changes `yarn integ test/aws-ecs-patterns/test/ec2/integ.multiple-application-load-balanced-ecs-service.js` ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent ec4b91b commit fd05c10

File tree

8 files changed

+659
-743
lines changed

8 files changed

+659
-743
lines changed

packages/@aws-cdk-testing/framework-integ/test/aws-ecs-patterns/test/ec2/integ.multiple-application-load-balanced-ecs-service.js.snapshot/applicationMultipleTargetGroupsEc2ServiceTestDefaultTestDeployAssert1194B50F.assets.json

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-ecs-patterns/test/ec2/integ.multiple-application-load-balanced-ecs-service.js.snapshot/aws-ecs-integ-multiple-alb.assets.json

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)