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

feat(ecs-patterns): Add Load Balancer name to ApplicationLoadBalancedFargateService props #14831

Merged
merged 6 commits into from
Jun 9, 2021
Merged

feat(ecs-patterns): Add Load Balancer name to ApplicationLoadBalancedFargateService props #14831

merged 6 commits into from
Jun 9, 2021

Conversation

richardhughes
Copy link
Contributor

@richardhughes richardhughes commented May 21, 2021

This PR allows the loadbalancer name to be passed to the ApplicationLoadBalancedFargateService. This PR just exposes a prop from the ApplicationLoadBalancer construct.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Add the loadbalancer name to the props so we can customise the LB name rather than having a random generated LB name
@gitpod-io
Copy link

gitpod-io bot commented May 21, 2021

@mergify
Copy link
Contributor

mergify bot commented May 21, 2021

Title does not follow the guidelines of Conventional Commits. Please adjust title before merge.

Add extra example to the readme to show the changes in the PR
Add a new test to cover the changes in the PR
@richardhughes richardhughes changed the title WIP: feat(ecs-patterns): Add LoadBalancer name to props feat(ecs-patterns): Add LoadBalancer name to props May 21, 2021
@richardhughes richardhughes marked this pull request as ready for review May 21, 2021 23:24
@richardhughes richardhughes changed the title feat(ecs-patterns): Add LoadBalancer name to props feat(ecs-patterns): Add Load Balancer name to ApplicationLoadBalancedFargateService props May 21, 2021
Add extra comma and remove back ticks
Fix the quotes and trailing commas
@felipeloha
Copy link

is there any way to work around this?

@@ -523,3 +523,21 @@ new QueueProcessingFargateService(stack, 'QueueProcessingService', {
image: new ecs.AssetImage(path.join(__dirname, '..', 'sqs-reader')),
});
```

### Select specific load balancer name ApplicationLoadBalancedFargateService
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Select specific load balancer name ApplicationLoadBalancedFargateService
### Specify load balancer name for an ApplicationLoadBalancedFargateService

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 1346de5
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify mergify bot merged commit c432fb4 into aws:master Jun 9, 2021
@mergify
Copy link
Contributor

mergify bot commented Jun 9, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@richardhughes richardhughes deleted the add-load-balancer-name-to-ecs-patterns branch June 9, 2021 09:37
hollanddd pushed a commit to hollanddd/aws-cdk that referenced this pull request Aug 26, 2021
…FargateService props (aws#14831)

This PR allows the loadbalancer name to be passed to the ApplicationLoadBalancedFargateService. This PR just exposes a prop from the ApplicationLoadBalancer construct.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants