Skip to content

Commit 5fe509b

Browse files
authored
Remove deprecated events package and methods (#45684)
* Remove deprecated events package and methods * Update CHANGELOG * Retrigger CI/CD pipeline * Retrigger CI/CD pipeline
1 parent 1c77f83 commit 5fe509b

3 files changed

Lines changed: 3 additions & 40 deletions

File tree

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
### Bugs Fixed
1010

1111
### Other Changes
12+
- Remove deprecated events packages.
13+
([#45684](https://github.com/Azure/azure-sdk-for-python/pull/45684))
1214

1315
## 1.8.7 (2026-03-20)
1416

@@ -36,6 +38,7 @@
3638

3739
### Other Changes
3840
- Add missing copyright headers and ensure consistent formatting across files.
41+
([#44908](https://github.com/Azure/azure-sdk-for-python/pull/44908))
3942

4043
## 1.8.5 (2026-01-28)
4144

sdk/monitor/azure-monitor-opentelemetry/azure/monitor/opentelemetry/_configure.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -267,21 +267,6 @@ def _setup_logging(configurations: Dict[str, ConfigurationValue]):
267267
)
268268
logger.addHandler(handler)
269269

270-
# Setup Events
271-
try:
272-
from opentelemetry._events import _set_event_logger_provider
273-
from opentelemetry.sdk._events import EventLoggerProvider
274-
275-
event_provider = EventLoggerProvider(logger_provider)
276-
_set_event_logger_provider(event_provider, False)
277-
except ImportError as ex:
278-
# If the events is not available, we will not set it up.
279-
# This could possibly be due to breaking change in upstream OpenTelemetry
280-
# Advise user to upgrade to latest OpenTelemetry version
281-
_logger.warning( # pylint: disable=do-not-log-exceptions-if-not-debug
282-
"Exception occurred when setting up Events. Please upgrade to the latest OpenTelemetry version: %s.",
283-
ex,
284-
)
285270
except ImportError as ex:
286271
# If the events is not available, we will not set it up.
287272
# This could possibly be due to breaking change in upstream OpenTelemetry

sdk/monitor/azure-monitor-opentelemetry/tests/test_configure.py

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -543,8 +543,6 @@ def test_setup_logging(self, get_logger_mock, pclp_mock):
543543
log_exporter_mock = Mock()
544544
blrp_mock = Mock()
545545
logging_handler_mock = Mock()
546-
elp_mock = Mock()
547-
set_elp_mock = Mock()
548546

549547
lp_init_mock = Mock()
550548
lp_mock.return_value = lp_init_mock
@@ -560,8 +558,6 @@ def test_setup_logging(self, get_logger_mock, pclp_mock):
560558
custom_lrp = Mock()
561559
get_logger_mock.return_value = logger_mock
562560
formatter_init_mock = Mock()
563-
elp_init_mock = Mock()
564-
elp_mock.return_value = elp_init_mock
565561
pclp_init_mock = Mock()
566562
pclp_mock.return_value = pclp_init_mock
567563
configurations = {
@@ -585,8 +581,6 @@ def test_setup_logging(self, get_logger_mock, pclp_mock):
585581
_AzureBatchLogRecordProcessor=blrp_mock
586582
),
587583
"azure.monitor.opentelemetry.exporter": Mock(AzureMonitorLogExporter=log_exporter_mock),
588-
"opentelemetry._events": Mock(_set_event_logger_provider=set_elp_mock),
589-
"opentelemetry.sdk._events": Mock(EventLoggerProvider=elp_mock),
590584
},
591585
):
592586
_setup_logging(configurations)
@@ -606,8 +600,6 @@ def test_setup_logging(self, get_logger_mock, pclp_mock):
606600
logging_handler_init_mock.setFormatter.assert_called_once_with(formatter_init_mock)
607601
get_logger_mock.assert_called_once_with("test")
608602
logger_mock.addHandler.assert_called_once_with(logging_handler_init_mock)
609-
elp_mock.assert_called_once_with(lp_init_mock)
610-
set_elp_mock.assert_called_once_with(elp_init_mock, False)
611603

612604
@patch("azure.monitor.opentelemetry._configure._PerformanceCountersLogRecordProcessor")
613605
@patch("azure.monitor.opentelemetry._configure.isinstance")
@@ -618,8 +610,6 @@ def test_setup_logging_duplicate_logger(self, get_logger_mock, instance_mock, pc
618610
set_logger_provider_mock = Mock()
619611
log_exporter_mock = Mock()
620612
blrp_mock = Mock()
621-
elp_mock = Mock()
622-
set_elp_mock = Mock()
623613

624614
# Create mock instances
625615
lp_init_mock = Mock()
@@ -640,9 +630,6 @@ def test_setup_logging_duplicate_logger(self, get_logger_mock, instance_mock, pc
640630
get_logger_mock.return_value = logger_mock
641631
instance_mock.return_value = True
642632

643-
elp_init_mock = Mock()
644-
elp_mock.return_value = elp_init_mock
645-
646633
configurations = {
647634
"connection_string": "test_cs",
648635
"enable_performance_counters": True,
@@ -663,8 +650,6 @@ def test_setup_logging_duplicate_logger(self, get_logger_mock, instance_mock, pc
663650
_AzureBatchLogRecordProcessor=blrp_mock
664651
),
665652
"azure.monitor.opentelemetry.exporter": Mock(AzureMonitorLogExporter=log_exporter_mock),
666-
"opentelemetry._events": Mock(_set_event_logger_provider=set_elp_mock),
667-
"opentelemetry.sdk._events": Mock(EventLoggerProvider=elp_mock),
668653
},
669654
):
670655
_setup_logging(configurations)
@@ -679,8 +664,6 @@ def test_setup_logging_duplicate_logger(self, get_logger_mock, instance_mock, pc
679664
get_logger_mock.assert_called_once_with("test")
680665
# The logger already has a LoggingHandler, so addHandler should not be called
681666
logger_mock.addHandler.assert_not_called()
682-
elp_mock.assert_called_once_with(lp_init_mock)
683-
set_elp_mock.assert_called_once_with(elp_init_mock, False)
684667

685668
@patch("azure.monitor.opentelemetry._configure._PerformanceCountersLogRecordProcessor")
686669
@patch("azure.monitor.opentelemetry._configure.getLogger")
@@ -690,8 +673,6 @@ def test_setup_logging_disable_performance_counters(self, get_logger_mock, pclp_
690673
log_exporter_mock = Mock()
691674
blrp_mock = Mock()
692675
logging_handler_mock = Mock()
693-
elp_mock = Mock()
694-
set_elp_mock = Mock()
695676

696677
lp_init_mock = Mock()
697678
lp_mock.return_value = lp_init_mock
@@ -706,8 +687,6 @@ def test_setup_logging_disable_performance_counters(self, get_logger_mock, pclp_
706687
logger_mock.handlers = []
707688
get_logger_mock.return_value = logger_mock
708689
formatter_init_mock = Mock()
709-
elp_init_mock = Mock()
710-
elp_mock.return_value = elp_init_mock
711690
pclp_init_mock = Mock()
712691
pclp_mock.return_value = pclp_init_mock
713692
configurations = {
@@ -731,8 +710,6 @@ def test_setup_logging_disable_performance_counters(self, get_logger_mock, pclp_
731710
_AzureBatchLogRecordProcessor=blrp_mock
732711
),
733712
"azure.monitor.opentelemetry.exporter": Mock(AzureMonitorLogExporter=log_exporter_mock),
734-
"opentelemetry._events": Mock(_set_event_logger_provider=set_elp_mock),
735-
"opentelemetry.sdk._events": Mock(EventLoggerProvider=elp_mock),
736713
},
737714
):
738715
_setup_logging(configurations)
@@ -747,8 +724,6 @@ def test_setup_logging_disable_performance_counters(self, get_logger_mock, pclp_
747724
logging_handler_init_mock.setFormatter.assert_called_once_with(formatter_init_mock)
748725
get_logger_mock.assert_called_once_with("test")
749726
logger_mock.addHandler.assert_called_once_with(logging_handler_init_mock)
750-
elp_mock.assert_called_once_with(lp_init_mock)
751-
set_elp_mock.assert_called_once_with(elp_init_mock, False)
752727

753728
@patch(
754729
"azure.monitor.opentelemetry._configure.enable_performance_counters",

0 commit comments

Comments
 (0)