-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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): allow customizing logdriver #3550
Conversation
@kohidave please review |
4a81267
to
3a34983
Compare
packages/@aws-cdk/aws-ecs-patterns/lib/base/load-balanced-service-base.ts
Outdated
Show resolved
Hide resolved
packages/@aws-cdk/aws-ecs-patterns/lib/base/queue-processing-service-base.ts
Outdated
Show resolved
Hide resolved
@@ -136,6 +160,11 @@ export abstract class QueueProcessingServiceBase extends Construct { | |||
constructor(scope: Construct, id: string, props: QueueProcessingServiceBaseProps) { | |||
super(scope, id); | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feels like we can probably reuse this logic across patterns (via common utility function)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Potentially, I'll leave this PR as is, and create an issue to discuss this, there's a lot of common functionality that's separated today.
See my title update |
c054059
to
00be984
Compare
Rebased with master |
Provide the ability to pass logdriver as a property of all ecs-patterns constructs and update all constructs to accept either a vpc or cluster.
Please read the contribution guidelines and follow the pull-request checklist.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license