Skip to content

Initialize OneSettings for Microsoft distro - #244

Merged
Leighton Chen (lzchen) merged 7 commits into
microsoft:mainfrom
lzchen:feature/onesettings-profile
Aug 11, 2026
Merged

Initialize OneSettings for Microsoft distro#244
Leighton Chen (lzchen) merged 7 commits into
microsoft:mainfrom
lzchen:feature/onesettings-profile

Conversation

@lzchen

Copy link
Copy Markdown
Contributor

Summary

  • initialize the OneSettings resource provider during Microsoft distro configuration
  • guard the private OneSettings import and type the optional factory
  • add coverage for initialization behavior and update dependency metadata/changelog

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07de762e-7d54-46cb-9d78-ab8ea1c82cbe
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07de762e-7d54-46cb-9d78-ab8ea1c82cbe
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07de762e-7d54-46cb-9d78-ab8ea1c82cbe
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07de762e-7d54-46cb-9d78-ab8ea1c82cbe
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07de762e-7d54-46cb-9d78-ab8ea1c82cbe
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07de762e-7d54-46cb-9d78-ab8ea1c82cbe
@github-actions

Copy link
Copy Markdown

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 19,910.8 19,764.8 +0.74%
azure_monitor_span yes 157,928.0 158,679.8 -0.47%
otel_log no 23,045.7 22,289.6 +3.39%
otel_span no 39,894.7 38,746.2 +2.96%

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 ensures the Microsoft OpenTelemetry Python distro contributes its identity to the shared OneSettings profile by initializing the Azure Monitor exporter’s configuration manager at the start of use_microsoft_opentelemetry(). It also updates the Azure Monitor exporter dependency and adds unit coverage around the new initialization behavior.

Changes:

  • Add guarded import + initialization hook for the Azure Monitor exporter configuration manager (OneSettings) in use_microsoft_opentelemetry().
  • Add tests validating initialization occurs (and occurs before exporter wiring) and that absence/disablement is handled.
  • Bump azure-monitor-opentelemetry-exporter to 1.0.0b56 and update changelog/lockfile + ignore VS Code settings.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/microsoft/opentelemetry/_distro.py Adds guarded access to the Azure Monitor configuration manager and initializes it early in use_microsoft_opentelemetry().
tests/test_distro.py Adds coverage for configuration manager initialization ordering and availability/disablement scenarios.
pyproject.toml Updates the Azure Monitor exporter dependency version constraint to ~=1.0.0b56.
uv.lock Updates the locked Azure Monitor exporter version/artifacts and corresponding dependency metadata.
CHANGELOG.md Documents the new OneSettings profile contribution behavior under the next unreleased version.
.gitignore Ignores .vscode/ workspace settings.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@lzchen
Leighton Chen (lzchen) merged commit 7b7f2b6 into microsoft:main Aug 11, 2026
22 of 23 checks passed
@lzchen
Leighton Chen (lzchen) deleted the feature/onesettings-profile branch August 11, 2026 17:59
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