Skip to content
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

(ecs-patterns): Failing integration test #32506

Open
1 task
phuhung273 opened this issue Dec 13, 2024 · 0 comments
Open
1 task

(ecs-patterns): Failing integration test #32506

phuhung273 opened this issue Dec 13, 2024 · 0 comments
Labels
@aws-cdk/aws-ecs-patterns Related to ecs-patterns library bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.

Comments

@phuhung273
Copy link
Contributor

phuhung273 commented Dec 13, 2024

Describe the bug

integ.multiple-application-load-balanced-ecs-service-idle-timeout

Error occurred: InvalidDomainNameException - example.com. is reserved by AWS!

integ.multiple-application-load-balanced-ecs-service

Behaviour: 1 out of 2 target groups always unhealthy

Try running sample image locally, port 90 doesn't work:
docker run -d -p 80:80 -p 90:90 amazon/amazon-ecs-sample:latest > curl http://localhost:90 FAIL
docker run -d -p 80:80 -p 90:80 amazon/amazon-ecs-sample:latest > curl http://localhost:90 WORK

Both tests are still using Launch Configuration > community member cannot run

Regression Issue

  • Select this option if this issue appears to be a regression.

Last Known Working CDK Version

No response

Expected Behavior

Integration tests should work seamlessly for anyone

Current Behavior

See description

Reproduction Steps

yarn integ test/aws-ecs-patterns/test/ec2/integ.multiple-application-load-balanced-ecs-service.js --force

Possible Solution

For both tests

  • Integration test should migrate to Launch Template instead of Launch Configuration

integ.multiple-application-load-balanced-ecs-service-idle-timeout

Use your own cert pattern

integ.multiple-application-load-balanced-ecs-service

I can only think of 2 solutions. None of them are clean. Would need team advice:

targetGroups: [
    {
      containerPort: 80,
      hostPort: 8080,
    },
    {
      containerPort: 90,
      hostPort: 8090,
      pathPattern: 'a/b/c',
      priority: 10,
    },
  ]

Additional Information/Context

No response

CDK CLI Version

main

Framework Version

No response

Node.js Version

v20.18.0

OS

Dev container

Language

TypeScript

Language Version

No response

Other information

No response

@phuhung273 phuhung273 added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Dec 13, 2024
@github-actions github-actions bot added the @aws-cdk/aws-ecs-patterns Related to ecs-patterns library label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-ecs-patterns Related to ecs-patterns library bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

1 participant