Change import path for `LoggingHandler` to accommodate upstream breaking changes by rads-1996 · Pull Request #45516 · Azure/azure-sdk-for-python

rads-1996 · 2026-03-04T20:51:48Z

Description

LoggingHandler from opentelemetry.sdk_logs is being deprecated. Need to use the LoggingHandler from the opentelemetry-instrumentation-logging instead.

All SDK Contribution checklist:

The pull request does not introduce [breaking changes]
CHANGELOG is updated for new features, bug fixes or other significant changes.
I have read the contribution guidelines.

General Guidelines and Best Practices

Title of the pull request is clear and informative.
There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Copilot

Pull request overview

Updates azure-monitor-opentelemetry-exporter samples/docs to use the LoggingHandler implementation from opentelemetry-instrumentation-logging (since opentelemetry.sdk._logs.LoggingHandler is being deprecated), and bumps pinned OpenTelemetry API/SDK versions accordingly.

Changes:

Bump opentelemetry-api and opentelemetry-sdk pins from 1.39 → 1.40.
Update README + log samples to import LoggingHandler from opentelemetry.instrumentation.logging.handler.
Add a changelog entry describing the import-path change.

Reviewed changes

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

Show a summary per file

File	Description
sdk/monitor/azure-monitor-opentelemetry-exporter/setup.py	Updates pinned OpenTelemetry API/SDK versions.
sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_properties.py	Switches `LoggingHandler` import to instrumentation package.
sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_log.py	Switches `LoggingHandler` import to instrumentation package.
sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_exception.py	Switches `LoggingHandler` import to instrumentation package.
sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_custom_event.py	Switches `LoggingHandler` import to instrumentation package.
sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_correlate.py	Switches `LoggingHandler` import to instrumentation package.
sdk/monitor/azure-monitor-opentelemetry-exporter/README.md	Updates code snippets to use new `LoggingHandler` import path.
sdk/monitor/azure-monitor-opentelemetry-exporter/CHANGELOG.md	Notes the import-path change for `LoggingHandler`.

You can also share your feedback on Copilot code review. Take the survey.

sdk/monitor/azure-monitor-opentelemetry-exporter/setup.py

sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_properties.py

sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_log.py

sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_exception.py

sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_custom_event.py

sdk/monitor/azure-monitor-opentelemetry-exporter/samples/logs/sample_correlate.py

sdk/monitor/azure-monitor-opentelemetry-exporter/README.md

sdk/monitor/azure-monitor-opentelemetry-exporter/CHANGELOG.md

rads-1996 · 2026-03-06T18:21:35Z

/check-enforcer override

rads-1996 · 2026-03-06T18:21:55Z

/check-enforcer override

rads-1996 · 2026-03-06T18:22:22Z

/check-enforcer override

sdk/monitor/azure-monitor-opentelemetry-exporter/CHANGELOG.md

…am OTEL

rads-1996 · 2026-03-06T19:00:00Z

/check-enforcer override

rads-1996 · 2026-03-06T19:40:11Z

/check-enforcer override

rads-1996 · 2026-03-06T19:44:21Z

/check-enforcer override

JacksonWeber

LGTM

rads-1996 · 2026-03-17T20:15:41Z

/check-enforcer override

rads-1996 · 2026-03-17T20:18:26Z

/check-enforcer override

rads-1996 · 2026-03-18T15:05:11Z

/check-enforcer override

rads-1996 · 2026-03-18T16:13:32Z

/check-enforcer override

rads-1996 · 2026-03-18T16:13:45Z

/check-enforcer override

rads-1996 · 2026-03-18T17:01:21Z

/check-enforcer override

xiangyan99 · 2026-03-18T20:27:12Z

sdk/monitor/azure-monitor-opentelemetry-exporter/setup.py

-        "opentelemetry-api==1.39",
-        "opentelemetry-sdk==1.39",
+        "opentelemetry-api==1.40",
+        "opentelemetry-sdk==1.40",


This conflicts with the version in sdk/monitor/azure-monitor-opentelemetry

I will update the versions in azure-monitor-opentelemetry after the exporter is released. azure-monitor-opentelemetry depends on the exporter, so the versions have to updated there first before making changes in azure-monitor-opentelemetry

Can you update it now? It blocks the azure-core release.

Can you update it now? It blocks the azure-core release.

I am trying to release the exporter. Until that is released, the distro changes will fail, even if I update them. Give me some time, I am working on releasing the exporter.

@rads-1996 what is your alias? I'll reach out over teams. You can skip the failing check with a queue time build variable of Skip.AnalyzeDependencies with value of true to get your release out.

Is there a commit I can use to release the exporter? If I use the https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6030525, it fails because of the dependency mismatch, I am using the skip.test=true and BuildTargetingString = azure-monitor-opentelemetry-exporter

@xiangyan99 I have released the exporter and have another PR up for review which updates the otel versions in azure-monitor-opentelemetry - #45539. Once it is approved, I will merge the PR.
EDIT: The PR has been merged. This should unblock your release.

github-actions bot added Monitor - Distro Monitor OpenTelemetry Distro Monitor - Exporter Monitor OpenTelemetry Exporter labels Mar 4, 2026

rads-1996 force-pushed the logging-handler branch 4 times, most recently from 6da3ac7 to 170f2de Compare March 5, 2026 21:37

rads-1996 changed the title ~~Change import path for LoggingHandler to accomodate upstream breaking changes~~ Change import path for LoggingHandler to accommodate upstream breaking changes Mar 5, 2026

rads-1996 force-pushed the logging-handler branch from b57b1cc to c421227 Compare March 5, 2026 23:45

rads-1996 marked this pull request as ready for review March 6, 2026 00:17

rads-1996 requested review from JacksonWeber, MSNev, hectorhdzg, jeremydvoss and lzchen as code owners March 6, 2026 00:17

Copilot AI review requested due to automatic review settings March 6, 2026 00:17

Copilot started reviewing on behalf of rads-1996 March 6, 2026 00:18 View session

Copilot AI reviewed Mar 6, 2026

View reviewed changes

rads-1996 force-pushed the logging-handler branch 2 times, most recently from ec9bd0d to b64b5f7 Compare March 6, 2026 16:48

hectorhdzg reviewed Mar 6, 2026

View reviewed changes

sdk/monitor/azure-monitor-opentelemetry-exporter/CHANGELOG.md Outdated Show resolved Hide resolved

rads-1996 added 7 commits March 6, 2026 10:58

Modify import for LoggingHandler following breaking changes in upstre…

ca55d93

…am OTEL

Updated README and setup.py

d9b8aa8

Update CHANGELOG

4eda687

Update sdk version

6f4cb1e

Fix dep issue

299af3e

Revert doc change to check if CI/CD passes

732ea30

Fix dep in exporter

06cba31

rads-1996 added 7 commits March 6, 2026 10:58

Move distro changes out to another PR

264e28d

Add back comma

9d602af

Update CHANGELOG

716d2b2

fix dep

a001fd2

Retrigger CI/CD pipeline

d73c432

Retrigger CI/CD pipeline

5fb7ae9

Update CHANGELOG

3000f72

rads-1996 force-pushed the logging-handler branch from 46f9601 to 3000f72 Compare March 6, 2026 18:59

hectorhdzg approved these changes Mar 6, 2026

View reviewed changes

Merge branch 'main' into logging-handler

cf836a0

rads-1996 mentioned this pull request Mar 6, 2026

Exporter release 1.0.0b49 #45558

Merged

JacksonWeber approved these changes Mar 9, 2026

View reviewed changes

Merge branch 'main' into logging-handler

7aa0483

Merge branch 'main' into logging-handler

52ba7c8

rads-1996 enabled auto-merge (squash) March 18, 2026 15:56

rads-1996 disabled auto-merge March 18, 2026 15:57

Merge branch 'main' into logging-handler

d8dcc5d

rads-1996 merged commit 63f2acb into Azure:main Mar 18, 2026
17 of 20 checks passed

xiangyan99 reviewed Mar 18, 2026

View reviewed changes

rads-1996 deleted the logging-handler branch March 19, 2026 15:13

Conversation

rads-1996 commented Mar 4, 2026

Description

All SDK Contribution checklist:

General Guidelines and Best Practices

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!

rads-1996 commented Mar 6, 2026

Uh oh!

rads-1996 commented Mar 6, 2026

Uh oh!

rads-1996 commented Mar 6, 2026

Uh oh!

Uh oh!

rads-1996 commented Mar 6, 2026

Uh oh!

rads-1996 commented Mar 6, 2026

Uh oh!

rads-1996 commented Mar 6, 2026

Uh oh!

JacksonWeber left a comment

Choose a reason for hiding this comment

Uh oh!

rads-1996 commented Mar 17, 2026

Uh oh!

rads-1996 commented Mar 17, 2026

Uh oh!

rads-1996 commented Mar 18, 2026

Uh oh!

rads-1996 commented Mar 18, 2026

Uh oh!

rads-1996 commented Mar 18, 2026

Uh oh!

rads-1996 commented Mar 18, 2026

Uh oh!

Uh oh!

xiangyan99 Mar 18, 2026

Choose a reason for hiding this comment

Uh oh!

rads-1996 Mar 18, 2026

Choose a reason for hiding this comment

Uh oh!

xiangyan99 Mar 18, 2026

Choose a reason for hiding this comment

Uh oh!

rads-1996 Mar 18, 2026

Choose a reason for hiding this comment

Uh oh!

scbedd Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rads-1996 Mar 18, 2026

Choose a reason for hiding this comment

Uh oh!

rads-1996 Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

scbedd Mar 18, 2026 •

edited

Loading

rads-1996 Mar 19, 2026 •

edited

Loading