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

option for LoadBalancedFargateService with NLB to specify different listener port #3666

Closed
1 of 5 tasks
pahud opened this issue Aug 15, 2019 · 1 comment
Closed
1 of 5 tasks
Assignees
Labels
@aws-cdk/aws-ecs Related to Amazon Elastic Container @aws-cdk/aws-ecs-patterns Related to ecs-patterns library

Comments

@pahud
Copy link
Contributor

pahud commented Aug 15, 2019

Note: for support questions, please first reference our documentation, then use Stackoverflow. This repository's issues are intended for feature requests and bug reports.

  • I'm submitting a ...

    • 🪲 bug report
    • 🚀 feature request
    • 📚 construct library gap
    • ☎️ security issue or vulnerability => Please see policy
    • ❓ support request => Please see note at the top of this template.
  • What is the current behavior?
    If the current behavior is a 🪲bug🪲: Please provide the steps to reproduce

The LoadBalancedFargateService class has no option to specify different ELB listerner port other than 80

export class LoadBalancedFargateService extends LoadBalancedServiceBase {

And we need to addPropertyOverride() like this:

圖片

  • What is the expected behavior (or behavior of feature suggested)?

Provide a loadBalancerListenerPort property in LoadBalancedFargateServiceProps

  • What is the motivation / use case for changing the behavior or adding this feature?

Fargate service running as TCP service behind NLB could have listener other than 80 on the NLB. For example, NLB:8888 to containerPort:8888 on Fargate.

  • Please tell us about your environment:

    • CDK CLI Version: 1.4.0 (build 175471f)
    • Module Version: 1.4.0
    • OS: OSX Mojave
    • Language: TypeScript
  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. associated pull-request, stackoverflow, gitter, etc)

@pahud pahud added the needs-triage This issue or PR still needs to be triaged. label Aug 15, 2019
@piradeepk piradeepk added the @aws-cdk/aws-ecs Related to Amazon Elastic Container label Aug 15, 2019
@SoManyHs SoManyHs added the @aws-cdk/aws-ecs-patterns Related to ecs-patterns library label Aug 19, 2019
@piradeepk piradeepk removed the needs-triage This issue or PR still needs to be triaged. label Aug 27, 2019
@SoManyHs
Copy link
Contributor

SoManyHs commented Oct 3, 2019

This should be addressed in #4213, please let us know if this works for you! @pahud

@SoManyHs SoManyHs closed this as completed Oct 3, 2019
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 @aws-cdk/aws-ecs-patterns Related to ecs-patterns library
Projects
None yet
Development

No branches or pull requests

3 participants