-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
DOCS-9940: Serverless Monitoring for Azure Functions #27294
base: master
Are you sure you want to change the base?
Conversation
Preview links (active after the
|
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.
Thanks for getting this started @cswatt! Let me know what you think about my suggestions.
Can we mention that we support runtime metrics and custom metrics and link the relevant documentation?
- https://docs.datadoghq.com/tracing/metrics/runtime_metrics/python/
- https://docs.datadoghq.com/tracing/metrics/runtime_metrics/nodejs
- https://docs.datadoghq.com/metrics/custom_metrics/dogstatsd_metrics_submission/
Could we also have more section (maybe under troubleshooting?) to document that if deploying to a Linux Consumption function with a Github Action that the "Using Azure Service Principal for RBAC as Deployment Credential" method needs to be used?
### Correlate logs and traces | ||
|
||
<!-- TODO: to be continued --> |
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 we can drop this for now while we work on more logging capabilities
Co-authored-by: Duncan Harvey <[email protected]>
patch_all() | ||
``` | ||
|
||
3. (Optional) **Enable runtime metrics**. See [Node.js Runtime Metrics][2]. |
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.
3. (Optional) **Enable runtime metrics**. See [Node.js Runtime Metrics][2]. | |
3. (Optional) **Enable runtime metrics**. See [Python Runtime Metrics][2]. |
@@ -0,0 +1,130 @@ | |||
--- | |||
title: Tracing Azure Functions |
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.
Maybe title should be Install Serverless Monitoring for Azure Functions
to be in line with https://docs.datadoghq.com/serverless/aws_lambda/installation?
What does this PR do? What is the motivation?
Merge instructions
Merge readiness:
Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the
<yourname>/description
naming convention) and then add the following PR comment:Additional notes