Skip to content

Using OTLPSpanExporter when configured through environment#188

Merged
nikhilNava merged 13 commits intomainfrom
users/robrandao/otlp-span-exporter-default
Mar 2, 2026
Merged

Using OTLPSpanExporter when configured through environment#188
nikhilNava merged 13 commits intomainfrom
users/robrandao/otlp-span-exporter-default

Conversation

@rodrigobr-msft
Copy link
Contributor

@rodrigobr-msft rodrigobr-msft commented Feb 27, 2026

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:

  • Added import for OTLPSpanExporter from opentelemetry.exporter.otlp.proto.http.trace_exporter to enable OTLP trace exporting.
  • Updated the _configure_internal method to check for the OTEL_EXPORTER_OTLP_ENDPOINT environment 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.

@rodrigobr-msft rodrigobr-msft changed the title Creating a OTLPSpanExporter if configured in environment Using a OTLPSpanExporter as fallback if configured in environment Feb 27, 2026
@github-actions
Copy link

github-actions bot commented Feb 27, 2026

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@rodrigobr-msft rodrigobr-msft changed the title Using a OTLPSpanExporter as fallback if configured in environment Using a OTLPSpanExporter as local fallback if configured in environment Feb 27, 2026
@rodrigobr-msft rodrigobr-msft marked this pull request as ready for review February 27, 2026 22:39
@rodrigobr-msft rodrigobr-msft requested a review from a team as a code owner February 27, 2026 22:39
Copilot AI review requested due to automatic review settings February 27, 2026 22:39
@rodrigobr-msft rodrigobr-msft changed the title Using a OTLPSpanExporter as local fallback if configured in environment Using a OTLPSpanExporter configured through environment Feb 27, 2026
@rodrigobr-msft rodrigobr-msft changed the title Using a OTLPSpanExporter configured through environment Using OTLPSpanExporter when configured through environment Feb 27, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 OTLPSpanExporter integration to configure an additional span processor when OTEL_EXPORTER_OTLP_ENDPOINT is set.
  • Added os import 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>
Copilot AI review requested due to automatic review settings February 27, 2026 22:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings February 27, 2026 23:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

@rodrigobr-msft rodrigobr-msft marked this pull request as draft February 27, 2026 23:13
@rodrigobr-msft rodrigobr-msft marked this pull request as ready for review February 27, 2026 23:15
Copilot AI review requested due to automatic review settings February 27, 2026 23:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

…gents_a365/observability/core/config.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 27, 2026 23:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

@nikhilNava nikhilNava merged commit a4f23f6 into main Mar 2, 2026
14 checks passed
@nikhilNava nikhilNava deleted the users/robrandao/otlp-span-exporter-default branch March 2, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants