Using OTLPSpanExporter when configured through environment - #188
Merged
Nikhil Navakiran (nikhilNava) merged 13 commits intoMar 2, 2026
Merged
Conversation
Rodrigo Brandão (rodrigobr-msft)
changed the title
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Rodrigo Brandão (rodrigobr-msft)
changed the title
Rodrigo Brandão (rodrigobr-msft)
marked this pull request as ready for review
February 27, 2026 22:39
Rodrigo Brandão (rodrigobr-msft)
requested a review
from a team
as a code owner
February 27, 2026 22:39
Copilot started reviewing on behalf of
Rodrigo Brandão (rodrigobr-msft)
February 27, 2026 22:39
View session
Rodrigo Brandão (rodrigobr-msft)
changed the title
Rodrigo Brandão (rodrigobr-msft)
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the observability core telemetry configuration to optionally add an OpenTelemetry OTLP HTTP trace exporter when OTLP environment configuration is detected, enabling export to a user-specified OTLP-compatible backend.
Changes:
- Added
OTLPSpanExporterintegration to configure an additional span processor whenOTEL_EXPORTER_OTLP_ENDPOINTis set. - Added
osimport to read OTLP environment configuration. - Minor warning-message formatting fix for the console-exporter fallback log.
…gents_a365/observability/core/config.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…github.com/microsoft/Agent365-python into users/robrandao/otlp-span-exporter-default
Rodrigo Brandão (rodrigobr-msft)
marked this pull request as draft
February 27, 2026 23:13
Rodrigo Brandão (rodrigobr-msft)
marked this pull request as ready for review
February 27, 2026 23:15
…gents_a365/observability/core/config.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…github.com/microsoft/Agent365-python into users/robrandao/otlp-span-exporter-default
MattB (MattB-msft)
approved these changes
Feb 28, 2026
Nikhil Navakiran (nikhilNava)
approved these changes
Mar 2, 2026
Nikhil Navakiran (nikhilNava)
deleted the
users/robrandao/otlp-span-exporter-default
branch
March 2, 2026 14:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the observability core configuration to support exporting telemetry data to an OTLP endpoint using environment variables. The main changes add support for the OTLP exporter, allowing the system to send traces to any OpenTelemetry-compatible backend if the relevant environment variable is set.
OTLP Exporter Integration:
OTLPSpanExporterfromopentelemetry.exporter.otlp.proto.http.trace_exporterto enable OTLP trace exporting._configure_internalmethod to check for theOTEL_EXPORTER_OTLP_ENDPOINTenvironment variable and, if present, configure and add an OTLP exporter as a span processor. This allows exporting traces to an OTLP-compatible endpoint based on environment configuration.