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(aws-ecs): Expose logdriver "mode" property #13965

Merged
merged 9 commits into from
May 4, 2021

Conversation

hassanazharkhan
Copy link
Contributor

Closes #13845


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

@gitpod-io
Copy link

gitpod-io bot commented Apr 2, 2021

@hassanazharkhan
Copy link
Contributor Author

@SoManyHs Can you please review this when you get a chance?

@gitpod-io
Copy link

gitpod-io bot commented May 2, 2021

Copy link
Contributor

@bvtujo bvtujo left a comment

Choose a reason for hiding this comment

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

Thank you so much for adding this property. I have just a small syntax suggestion to make the property definition in bind more congruent with the others, but other than that this looks good to me!

@SoManyHs SoManyHs added the @aws-cdk/aws-ecs Related to Amazon Elastic Container label May 4, 2021
Co-authored-by: Austin Ely <austiely@amazon.com>
bvtujo
bvtujo previously approved these changes May 4, 2021
Copy link
Contributor

@bvtujo bvtujo left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks for the quick turnaround!

@mergify mergify bot dismissed bvtujo’s stale review May 4, 2021 17:30

Pull request has been modified.

@hassanazharkhan
Copy link
Contributor Author

@bvtujo Could you please re-approve?

@mergify
Copy link
Contributor

mergify bot commented May 4, 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).

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 2c28c2d
  • 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 28fce22 into aws:master May 4, 2021
@mergify
Copy link
Contributor

mergify bot commented May 4, 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).

john-tipper pushed a commit to john-tipper/aws-cdk that referenced this pull request May 10, 2021
Closes [aws#13845](aws#13845)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
hollanddd pushed a commit to hollanddd/aws-cdk that referenced this pull request Aug 26, 2021
Closes [aws#13845](aws#13845)

----

*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 Related to Amazon Elastic Container
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(@aws-cdk/aws-ecs): Expose logdriver "mode" property for @aws-cdk/aws-ecs.AwsLogDriverProps
4 participants