-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
inference tracing #31160
base: main
Are you sure you want to change the base?
inference tracing #31160
Conversation
sdk/instrumentation/opentelemetry-instrumentation-azure-sdk/src/spanWrapper.ts
Outdated
Show resolved
Hide resolved
API change check API changes are not detected in this pull request. |
f5bbe48
to
98fddbc
Compare
4f0b161
to
c162981
Compare
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.
Left some minor comments, but tracing-related changes LGTM overall!
dc491f3
to
23baa68
Compare
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 working with us here, and I appreciate your thorough end-to-end validation 👍
Packages impacted by this PR
ai-inference
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Yes. added the test and recording.
Provide a list of related PRs (if any)
Nothing
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists