Add Microsoft Opentelemetry Distro Configuration by JacksonWeber · Pull Request #9 · microsoft/opentelemetry-distro-python

Jackson Weber (JacksonWeber) · 2026-04-01T20:20:59Z

This pull request introduces the initial implementation of the microsoft.opentelemetry distribution package, providing a unified configuration entry point for OpenTelemetry telemetry collection and export via Azure Monitor, OTLP, and A365. The most significant changes include the addition of the main configuration API, supporting utilities, and comprehensive tests to ensure correct integration and error handling.

New Microsoft OpenTelemetry Distribution:

Introduced the microsoft.opentelemetry package with a public configure_microsoft_opentelemetry() entry point that orchestrates configuration for Azure Monitor, OTLP, and A365 export paths. This function handles parameter remapping and delegates setup to the azure-monitor-opentelemetry package when appropriate. [1] [2] [3]

Configuration and Utilities:

Added configuration helpers and constants to manage Microsoft-specific options, including environment variable handling, default values, and remapping of connection string parameters. These utilities ensure seamless integration with both Microsoft and Azure Monitor configuration conventions. [1] [2] [3] [4] [5]

Testing and Validation:

Added comprehensive unit and integration tests to validate configuration logic, public API surface, parameter remapping, environment variable handling, and error paths (such as missing dependencies). Tests ensure that only the correct parameters are forwarded to Azure Monitor and that all Microsoft-specific logic is exercised. [1] [2]

Adds the microsoft.opentelemetry package that wraps azure-monitor-opentelemetry with configure_microsoft_opentelemetry() entry point, supporting Azure Monitor, OTLP, and A365 export paths. Includes unit tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Tests cover: public API surface, standalone provider setup (tracing, logging, metrics), signal enable/disable flags, OTLP metric reader and exporter config, A365 exporter error handling, Azure Monitor import errors, TracerProvider unwrapping, environment variable configuration, and end-to-end standalone mode. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Remove all OTLP, A365, and GenAI exporter/instrumentation code. This PR now adds only the Azure Monitor configuration path and standalone provider setup. OTLP and other export paths will be added in future PRs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Remove standalone provider setup, instrumentation setup, and helper functions that duplicated what configure_azure_monitor() already handles. The distro now cleanly delegates to azure-monitor-opentelemetry for all provider, exporter, and instrumentation setup. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

Copilot

Pull request overview

This PR introduces a new microsoft.opentelemetry distro-style package intended to provide a single configure_microsoft_opentelemetry() entry point, delegating Azure Monitor setup to azure-monitor-opentelemetry, along with initial unit/integration tests and a changelog update.

Changes:

Added microsoft.opentelemetry package with configuration orchestration and Azure Monitor delegation/kwargs remapping.
Added configuration-building utilities/constants/types and a package version constant.
Added new tests covering API surface, env var behavior, and Azure Monitor delegation/error paths.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 12 comments.

Show a summary per file

File	Description
microsoft/opentelemetry/init.py	Exposes `configure_microsoft_opentelemetry` and `__version__`.
microsoft/opentelemetry/_configure.py	Implements orchestration and delegates to `azure.monitor.opentelemetry.configure_azure_monitor`.
microsoft/opentelemetry/_utils/configurations.py	Builds configuration dict using shared defaults and Microsoft-specific defaults/warnings.
microsoft/opentelemetry/_constants.py	Defines Microsoft-specific configuration keys and re-exports shared constants.
microsoft/opentelemetry/_types.py	Re-exports shared typing (`ConfigurationValue`).
microsoft/opentelemetry/_version.py	Introduces an internal VERSION constant.
microsoft/opentelemetry/_utils/init.py	Adds package marker file.
microsoft/init.py	Introduces top-level `microsoft` package marker.
microsoft/tests/test_configure.py	Adds tests for delegation/remapping/filtering behavior.
microsoft/tests/test_integration.py	Adds tests for API surface, env var handling, and import error warnings.
CHANGELOG.md	Documents the new distro configuration entry point (currently overclaims export-path support).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

The microsoft distro's _get_configurations() was re-importing and calling all 16+ _default_* functions from azure-monitor-opentelemetry, but configure_azure_monitor() already calls its own _get_configurations() internally — so all defaults were applied twice. Simplified configure_microsoft_opentelemetry() to only handle the two microsoft-specific concerns: - Connection string resolution and key remapping (azure_monitor_connection_string -> connection_string) - The enable_azure_monitor_export gate (making Azure Monitor optional) All other configuration defaults (resource, sampling, instrumentations, processors, etc.) are now handled solely by configure_azure_monitor(). Removed microsoft/opentelemetry/_utils/ (no longer needed). Updated tests to match simplified flow and fixed pre-existing mock issue with create=True for namespace package patching. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…not using az mon exporter.

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

- Remove module-level import of azure.monitor.opentelemetry._constants in _constants.py (caused ModuleNotFoundError in CI) - Define ConfigurationValue locally in _types.py instead of re-importing from azure.monitor.opentelemetry._types - Fix line-too-long pylint warnings in _configure.py and _constants.py - Add pylint inline disables for intentional patterns (import-outside-toplevel, broad-exception-caught) - Auto-format test files with black Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- test_configure.py imports opentelemetry.sdk.resources.Resource which requires opentelemetry-sdk at test time - Update pr-validation.yml to run black/pylint on microsoft/tests/ too Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Rename enable_azure_monitor_export to disable_azure_monitor_exporter with inverted boolean logic (exporter enabled by default) - Add disable_live_metrics and disable_performance_counters kwargs with _remap_disable_to_enable() converting to the enable_* form expected by configure_azure_monitor() - Update README with current API example and config options table - Update CHANGELOG version to 0.1.0b1 with new entries - Add 3 new tests for disable-to-enable remapping (25 total) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@patch

- Rewrite TestSetupAzureMonitor tests to mock azure.monitor.opentelemetry via sys.modules instead of @patch(create=True), which fails when the azure package is not installed - Fix implicit-str-concat in test_integration.py - Add import-error, no-name-in-module, import-outside-toplevel to global pylint disable list (test-time and optional deps) - Shorten docstrings to fit line-length limit Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

Move Azure Monitor OpenTelemetry modules from azure/monitor/opentelemetry/ to microsoft/azureMonitor/ namespace. Update configure module and tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Bring upstream azure-monitor-opentelemetry tests into the repo organized by scenario per reviewer feedback: - tests/ (root): Microsoft Distro tests - tests/azureMonitor/: Azure Monitor specific tests - autoinstrumentation, browserSdkLoader, diagnostics, utils All imports adapted from azure.monitor.opentelemetry to microsoft.azureMonitor. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…able Rename Azure Monitor-specific kwargs in use_microsoft_opentelemetry(): - connection_string -> azure_monitor_connection_string - credential -> azure_monitor_exporter_credential - enable_live_metrics -> azure_monitor_enable_live_metrics - enable_performance_counters -> azure_monitor_enable_performance_counters - disable_offline_storage -> azure_monitor_exporter_disable_offline_storage - storage_directory -> azure_monitor_exporter_storage_directory - browser_sdk_loader_config -> azure_monitor_browser_sdk_loader_config - disable_azure_monitor_exporter -> enable_azure_monitor (inverted) Add comprehensive test covering all config options end-to-end. Remove outdated Current Repository Layout section from README. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Update CHANGELOG.md with current kwarg names. Update PLANNING.md references from configure_microsoft_opentelemetry to use_microsoft_opentelemetry. Fix broken absolute paths in README.md links. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…-distro-config # Conflicts: # CHANGELOG.md # PLANNING.md # README.md # pyproject.toml # samples/langchain/sample_opentelemetry.py

…m/JacksonWeber/opentelemetry-distro-python into jacksonweber/add-ms-distro-config

…annotations - Add namespace_packages, explicit_package_bases, mypy_path to mypy config to fix 'Source file found twice under different module names' error - Exclude src/microsoft/azureMonitor from mypy (same as pylint) - Add py.typed marker for microsoft.opentelemetry package - Add type annotations to use_microsoft_opentelemetry and _setup_azure_monitor Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Same exclusion as pylint and mypy — these vendored files depend on azure-monitor-opentelemetry and other packages not in dev deps. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Add opentelemetry-sdk, opentelemetry-api, opentelemetry-instrumentation, azure-monitor-opentelemetry-exporter, and azure-core as dependencies - Remove blanket exclude for src/microsoft/azureMonitor from mypy/pyright - Use mypy per-module overrides to relax type annotation requirements for vendored azureMonitor code and optional django dependency - Add specific pylint message disables for conventions common in vendored code (too-many-args, missing-class-docstring, etc.) - Keep tests/azureMonitor excluded from pylint and pyright (test files that require additional runtime deps not in dev extras) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Leftover stub files with no code and no imports referencing them. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…umentation These packages don't ship py.typed markers or type stubs, so mypy reports import-not-found. Add per-module ignore_missing_imports. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Pyright doesn't support per-module overrides like mypy. The azureMonitor code has missing type stubs (django, azure.core) and type expression issues that can't be configured away — use pyright's ignore for these vendored files while still type-checking microsoft.opentelemetry. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 50 out of 51 changed files in this pull request and generated 14 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Hector Hernandez (hectorhdzg)

We may need to work on top of this, but let's merge so we can work in parallel

Jackson Weber (JacksonWeber) and others added 5 commits April 1, 2026 13:19

Remove unused constants, utils, and re-exports

912a248

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

Jackson Weber (JacksonWeber) marked this pull request as ready for review April 1, 2026 22:49

Jackson Weber (JacksonWeber) requested review from Copilot, Hector Hernandez (hectorhdzg) and Radhika Gupta (rads-1996) April 1, 2026 22:49

Update changelog to reflect Azure Monitor only scope

d6c7d4c

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

Copilot started reviewing on behalf of Jackson Weber (JacksonWeber) April 1, 2026 22:51 View session

Copilot AI reviewed Apr 1, 2026

View reviewed changes

Radhika Gupta (rads-1996) reviewed Apr 2, 2026

View reviewed changes

Comment thread microsoft/opentelemetry/_utils/configurations.py Outdated

Radhika Gupta (rads-1996) reviewed Apr 2, 2026

View reviewed changes

Comment thread microsoft/opentelemetry/_utils/configurations.py Outdated

Jackson Weber (JacksonWeber) changed the title ~~Add microsoft opentelemetry distro configuration~~ Add Microsoft Opentelemetry Distro Configuration Apr 3, 2026

Jackson Weber (JacksonWeber) added 4 commits April 3, 2026 11:31

Do not warn on no connection string, as this could be intentional if …

56df7a6

…not using az mon exporter.

Move files for import support.

e5024d2

Update pyproject.toml

4bc99d7

Final cleanup based on PR comments.

9b0591a

Radhika Gupta (rads-1996) reviewed Apr 3, 2026

View reviewed changes

Comment thread src/microsoft/opentelemetry/_constants.py Outdated

Radhika Gupta (rads-1996) reviewed Apr 3, 2026

View reviewed changes

Comment thread src/microsoft/opentelemetry/_configure.py Outdated

Radhika Gupta (rads-1996) reviewed Apr 3, 2026

View reviewed changes

Comment thread src/microsoft/opentelemetry/_configure.py Outdated

Radhika Gupta (rads-1996) reviewed Apr 3, 2026

View reviewed changes

Comment thread src/microsoft/opentelemetry/_configure.py Outdated

Jackson Weber (JacksonWeber) and others added 6 commits April 3, 2026 12:18

Merge origin/main, resolve pyproject.toml testpaths conflict

272c019

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

Move remap_disable_to_enable to _utils/configurations.py

a99504d

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

Hector Hernandez (hectorhdzg) reviewed Apr 5, 2026

View reviewed changes

Comment thread README.md Outdated

Hector Hernandez (hectorhdzg) reviewed Apr 5, 2026

View reviewed changes

Comment thread README.md Outdated

Hector Hernandez (hectorhdzg) reviewed Apr 5, 2026

View reviewed changes

Comment thread README.md Outdated

Radhika Gupta (rads-1996) reviewed Apr 6, 2026

View reviewed changes

Comment thread pyproject.toml

Radhika Gupta (rads-1996) reviewed Apr 6, 2026

View reviewed changes

Comment thread pyproject.toml

Restructure namespace from azure.monitor to microsoft.azureMonitor

8de4175

Move Azure Monitor OpenTelemetry modules from azure/monitor/opentelemetry/ to microsoft/azureMonitor/ namespace. Update configure module and tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Jackson Weber (JacksonWeber) requested review from Nev (MSNev) and Leighton Chen (lzchen) as code owners April 6, 2026 21:27

Jackson Weber (JacksonWeber) and others added 13 commits April 6, 2026 14:41

Merge remote-tracking branch 'upstream/main' into jacksonweber/add-ms…

6ac84d1

…-distro-config # Conflicts: # CHANGELOG.md # PLANNING.md # README.md # pyproject.toml # samples/langchain/sample_opentelemetry.py

Merge remote-tracking branch 'upstream/main' into jacksonweber/add-ms…

aa32760

…-distro-config # Conflicts: # CHANGELOG.md # PLANNING.md # README.md # pyproject.toml # samples/langchain/sample_opentelemetry.py

Merge branch 'jacksonweber/add-ms-distro-config' of https://github.co…

142ce50

…m/JacksonWeber/opentelemetry-distro-python into jacksonweber/add-ms-distro-config

Update pyproject.toml

8c0a1b7

Exclude src/microsoft/azureMonitor from pyright

97d23af

Same exclusion as pylint and mypy — these vendored files depend on azure-monitor-opentelemetry and other packages not in dev deps. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Remove empty src/microsoft/opentelemetry/_utils/ directory

cc1f580

Leftover stub files with no code and no imports referencing them. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fix mypy: add import overrides for azure.core and opentelemetry.instr…

2bfe24b

…umentation These packages don't ship py.typed markers or type stubs, so mypy reports import-not-found. Add per-module ignore_missing_imports. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Jackson Weber (JacksonWeber) requested review from Copilot, Hector Hernandez (hectorhdzg) and Radhika Gupta (rads-1996) April 6, 2026 23:48

Copilot started reviewing on behalf of Jackson Weber (JacksonWeber) April 6, 2026 23:49 View session

Copilot AI reviewed Apr 6, 2026

View reviewed changes

Hector Hernandez (hectorhdzg) approved these changes Apr 7, 2026

View reviewed changes

Leighton Chen (lzchen) approved these changes Apr 7, 2026

View reviewed changes

Jackson Weber (JacksonWeber) merged commit 1ecad4f into microsoft:main Apr 7, 2026
10 checks passed

Leighton Chen (lzchen) deleted the jacksonweber/add-ms-distro-config branch April 7, 2026 00:19

Conversation

Jackson Weber (JacksonWeber) commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Hector Hernandez (hectorhdzg) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Jackson Weber (JacksonWeber) commented Apr 1, 2026 •

edited

Loading