Initialize OneSettings for Microsoft distro - #244
Merged
Leighton Chen (lzchen) merged 7 commits intoAug 11, 2026
Conversation
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
This reverts commit bafe402.
Leighton Chen (lzchen)
requested a review
from Hector Hernandez (hectorhdzg)
as a code owner
August 11, 2026 17:23
Leighton Chen (lzchen)
requested review from
Jackson Weber (JacksonWeber),
Nev (MSNev),
Jeremy Voss (jeremydvoss) and
Radhika Gupta (rads-1996)
as code owners
August 11, 2026 17:23
Performance comparisonThreshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.
|
Contributor
There was a problem hiding this comment.
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-exporterto1.0.0b56and 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.
Radhika Gupta (rads-1996)
approved these changes
Aug 11, 2026
Radhika Gupta (rads-1996)
approved these changes
Aug 11, 2026
Jackson Weber (JacksonWeber)
approved these changes
Aug 11, 2026
Leighton Chen (lzchen)
merged commit Aug 11, 2026
7b7f2b6
into
microsoft:main
22 of 23 checks passed
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.
Summary