Skip to content

Commit

Permalink
chore(ecs-patterns): fix ec2/multiple-application-load-balanced-ecs-s…
Browse files Browse the repository at this point in the history
…ervice 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*
  • Loading branch information
phuhung273 authored Feb 7, 2025
1 parent ec4b91b commit fd05c10
Show file tree
Hide file tree
Showing 8 changed files with 659 additions and 743 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit fd05c10

Please sign in to comment.