-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[MLOB-2380] Add LLM Observability Serverless Quickstart Guide #28118
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
[MLOB-2380] Add LLM Observability Serverless Quickstart Guide #28118
Conversation
Co-authored-by: cecilia saixue watt <cecilia.watt@datadoghq.com>
Co-authored-by: cecilia saixue watt <cecilia.watt@datadoghq.com>
Co-authored-by: cecilia saixue watt <cecilia.watt@datadoghq.com>
waiting on the next release of |
export DATADOG_API_KEY_SECRET_ARN=<DATADOG_API_KEY_SECRET_ARN> | ||
``` | ||
4. Instrument your Lambda function with LLM Observability (this requires at least version 77 of the Datadog Extension layer). | ||
{{< tabs >}} |
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.
Because this tabbed section is nested inside a numbered list, the tabs are indented. I think it looks a bit odd, but perhaps it is intentional and desired? Take a look and tell me what you think. If you'd like the tabs to be left justified, I think adding some extra newlines will separate them from the numbered list.
To see what I mean about the indentation, go to the preview, search on the page for the string "77", and look at the tabs just below the search result.
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.
ahh yes i see what you're saying! yeah let me try adding some newlines around it to see if it helps a bit. it is intentional to have a tab section here, although the misalignment wasn't intentional, but otherwise i don't think it's something we're too concerned about.
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 tried a bunch of newline configurations and couldn't quite get it to look right. any ideas for how to get the spacing a little better are appreciated 😄 we wanted the tabs here because of the different layer versions for the different language extensions. thanks!!
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 also played around with newlines. I was able to get the tab left justified, but it looked strange. I think the current configuration is the best. Thanks for being willing to experiment!
Co-authored-by: Ursula Chen <58821586+urseberry@users.noreply.github.com>
/merge |
View all feedbacks in Devflow UI.
The expected merge time in
|
What does this PR do? What is the motivation?
Adds a quickstart guide for LLM Observability in serverless.
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