-
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): add additional log drivers #3762
Conversation
Pull Request Checklist
|
@eladb It wasn't totally clear to me if there was a different way to implement the journald driver, but if there is feel free to let me know and we can close this PR and related issue. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
I'm not entirely sure the travis check is failing because of my changes. I think it's related to something else. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
'fluentd-address': this.props.address, | ||
'fluentd-async-connect': this.props.asyncConnect, | ||
'fluentd-buffer-limit': this.props.bufferLimit, | ||
'fluentd-retry-wait': this.props.retryWait, |
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.
Unfortunately, you can't just put a Duration into a response object. You'll have to convert it to seconds/milliseconds whatever using the right function on the Duration object.
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.
Yea i saw that in the error. i am nearly done with an update to fix this.
Continuous integration build failed |
Thanks so much for taking the time to contribute to the AWS CDK ❤️ We will shortly assign someone to review this pull request and help get it
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
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.
I approve, but manually merged to preserve the commit message. do-not-merge is not working ATM.
Rationale : Adds journald LogDriver to ECS.
Issues: closes #3761
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license