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: allow use of existing load balancer in Load Balanced Services #3737

Closed
2 tasks
bgdnlp opened this issue Aug 21, 2019 · 3 comments
Closed
2 tasks
Assignees
Labels
@aws-cdk/aws-ecs Related to Amazon Elastic Container feature-request A feature should be added or improved.

Comments

@bgdnlp
Copy link

bgdnlp commented Aug 21, 2019

🚀 Feature Request

General Information

  • 👋 I may be able to implement this feature request
  • ⚠️ This feature might incur a breaking change

Description

Would like to be able to use an existing load balancer with LoadBalancedFargateService and LoadBalancedEc2Service.

Load balancer pricing includes a fee per hour of load balancer running, which currently means about $18 per month. Running many services, each with its own balancer, can quickly add up.

Additionally, creating a separate load balancer for each service means we can't run multiple services under the same DNS name. So it's not possible to call different microservices depending on the path, for example (probably a common pattern?). Or group multiple subdomains on the same load balancer.

Proposed Solution

Allow passing an existing load balancer as a parameter, the same way it is now possible to specify an existing vpc or cluster.

If an existing load balancer is used, passing any parameter that changes the behavior of the load balancer ( #3691 ) should probably return an error or be ignored, as it might interfere with the behavior of other services. Adding a new listener, or adding a target group to an existing listener may be ok, but changing subnets for an existing load balancer probably shouldn't be done here.

Environment

  • **CDK CLI Version:1.4.0
  • **Module Version:1.4.0
  • **OS:all
  • **Language:all

Other information

@bgdnlp bgdnlp added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Aug 21, 2019
@piradeepk piradeepk added @aws-cdk/aws-ecs Related to Amazon Elastic Container and removed needs-triage This issue or PR still needs to be triaged. labels Aug 21, 2019
@piradeepk
Copy link
Contributor

@bgdnlp Thanks for the feature request! We're looking into the request and will get back to you once we have more information to provide.

@piradeepk
Copy link
Contributor

Resolved as part of #4213

@ggallotti
Copy link

Could we reuse ALB with ApplicationMultipleTargetGroupsEc2Service?

So we can have multiples ApplicationMultipleTargetGroupsEc2Service using the same Load Balancer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-ecs Related to Amazon Elastic Container feature-request A feature should be added or improved.
Projects
None yet
Development

No branches or pull requests

4 participants