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
Expose DD_SERVERLESS_LOGS_ENABLED env var, bump deps #146
Conversation
Codecov Report
@@ Coverage Diff @@
## master #146 +/- ##
==========================================
+ Coverage 79.77% 79.88% +0.11%
==========================================
Files 10 10
Lines 529 532 +3
Branches 121 122 +1
==========================================
+ Hits 422 425 +3
Misses 74 74
Partials 33 33
Continue to review full report at Codecov.
|
Looks great so far. I think all that's missing is documenting |
"check-formatting": "prettier --check src/**", | ||
"format": "prettier --write src/**" | ||
"check-formatting": "prettier --check \"src/**\" \"README.md\"", | ||
"format": "prettier --write \"src/**\" \"README.md\"" |
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.
👍
README.md
Outdated
| `logLevel` | The log level, set to `DEBUG` for extended logging. | | ||
| `enableXrayTracing` | Set `true` to enable X-Ray tracing on the Lambda functions and API Gateway integrations. Defaults to `false`. | | ||
| `enableDDTracing` | Enable Datadog tracing on the Lambda function. Defaults to `true`. | | ||
| `enableDDLogs` | Enable Datadog log collection for the Lambda function. Defaults to `true`. | |
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.
Does this apply when using the Datadog Forwarder?
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 think the answer to my own question is "no", that is, this setting only affects log collection if the customer is using the Lambda Extension, and not the older but still more popular Datadog Forwarder-based approach. I think we should document that here.
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 think you're correct, it doesn't apply to the Datadog Forwarder. Good call on documenting that here, will do
What does this PR do?
yarn audit --fix
enableDDLogs
in config andDD_SERVERLESS_LOGS_ENABLED
in environment:(I also verified that it's false when specified)
Motivation
Testing Guidelines
Additional Notes
Types of changes
Check all that apply