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

Example Network loadbalancer #4794

Closed
davidsteed opened this issue Oct 31, 2019 · 1 comment · Fixed by aws-samples/aws-cdk-examples#155
Closed

Example Network loadbalancer #4794

davidsteed opened this issue Oct 31, 2019 · 1 comment · Fixed by aws-samples/aws-cdk-examples#155
Assignees
Labels
@aws-cdk/aws-ecs-patterns Related to ecs-patterns library bug This issue is a bug. in-progress This issue is being actively worked on.

Comments

@davidsteed
Copy link

davidsteed commented Oct 31, 2019

The example https://github.com/aws-samples/aws-cdk-examples/tree/master/typescript/ecs/fargate-load-balanced-service/

does not work

It used to in V1.5.0, but the network load balancer no longer has a security group attached with ingres and egress properties (it used to in V1.5)

Reproduction Steps

export class NetworkloadbalancerStack extends cdk.Stack {
  constructor(scope: cdk.Construct, id: string, props?: cdk.StackProps) {
    super(scope, id, props);
    const vpc = new ec2.Vpc(this, 'MyVpc', { maxAzs: 2 });
    const cluster = new ecs.Cluster(this, 'Cluster', { vpc });

    // Instantiate Fargate Service with just cluster and image
    const fargateService = new ecs_patterns.NetworkLoadBalancedFargateService(this, "FargateService", {
      cluster,
      taskImageOptions: {
        image: ecs.ContainerImage.fromRegistry("amazon/amazon-ecs-sample"),
      },
    });

    // Output the DNS where you can access your service
    new cdk.CfnOutput(this, 'LoadBalancerDNS', { value: fargateService.loadBalancer.loadBalancerDnsName });
    // The code that defines your stack goes here
  }
}

Error Log

After creating the stack curl the url provided and there is no response

Environment

  • **CLI Version :1.15.0
  • **Framework Version:1.15.0
  • OS :
  • **Language :typescript

Other


This is 🐛 Bug Report

@davidsteed davidsteed added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Oct 31, 2019
@SomayaB SomayaB added the @aws-cdk/aws-ecs Related to Amazon Elastic Container label Oct 31, 2019
@piradeepk piradeepk added @aws-cdk/aws-ecs-patterns Related to ecs-patterns library and removed @aws-cdk/aws-ecs Related to Amazon Elastic Container labels Oct 31, 2019
@piradeepk piradeepk added in-progress This issue is being actively worked on. and removed needs-triage This issue or PR still needs to be triaged. labels Nov 12, 2019
@piradeepk
Copy link
Contributor

Resolved in: aws-samples/aws-cdk-examples#155

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. in-progress This issue is being actively worked on.
Projects
None yet
5 participants