-
Notifications
You must be signed in to change notification settings - Fork 942
Python: [BREAKING] Observability updates #2782
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
Python: [BREAKING] Observability updates #2782
Conversation
449f8e1 to
7f77071
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
python/samples/getting_started/observability/agent_with_foundry_tracing.py
Outdated
Show resolved
Hide resolved
…milar to AzureOpenAIChatClient Fixes microsoft#2186
b78af6b to
debdb5e
Compare
TaoChenOSU
left a comment
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 really like where we are going. After the PR is merged and before the next release, we need to update our documentation.
Motivation and Context
Made the setup a lot simpler, and more compatible:
configure_azure_monitor) that you run that, and then only callsetup_observabilitywhen you want to programmatically ensure that the code paths that emit traces, metrics, etc. are enabled. if you set the right env variables, then that is not needed.Description
Closes:
Contribution Checklist