Mark events API/SDK as deprecated

Author: DylanRussell

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+- Mark the Events API/SDK as deprecated. The Logs API/SDK should be used instead. An event is now a `LogRecord` with the `event_name` field set.
 - Update OTLP proto to v1.7 [#4645](https://github.com/open-telemetry/opentelemetry-python/pull/4645).
 - Update OTLP gRPC/HTTP exporters: the export timeout is now inclusive of all retries and backoffs.
   A +/-20% jitter was added to all backoffs. A pointless 32 second sleep that occurred after all retries

opentelemetry-api/src/opentelemetry/_events/__init__.py

@@ -17,6 +17,8 @@
 from os import environ
 from typing import Optional, cast
 
+from typing_extensions import deprecated
+
 from opentelemetry._logs import LogRecord
 from opentelemetry._logs.severity import SeverityNumber
 from opentelemetry.environment_variables import (
@@ -30,6 +32,9 @@
 _logger = getLogger(__name__)
 
 
+@deprecated(
+    "You should use `LogRecord` with the `event_name` field set instead. Deprecated since version 1.35.0."
+)
 class Event(LogRecord):
     def __init__(
         self,
@@ -59,6 +64,9 @@ def __init__(
         self.name = name
 
 
+@deprecated(
+    "You should use `Logger` instead. Deprecated since version 1.35.0."
+)
 class EventLogger(ABC):
     def __init__(
         self,
@@ -77,11 +85,17 @@ def emit(self, event: "Event") -> None:
         """Emits a :class:`Event` representing an event."""
 
 
+@deprecated(
+    "You should use `NoOpLogger` instead. Deprecated since version 1.35.0."
+)
 class NoOpEventLogger(EventLogger):
     def emit(self, event: Event) -> None:
         pass
 
 
+@deprecated(
+    "You should use `ProxyLogger` instead. Deprecated since version 1.35.0."
+)
 class ProxyEventLogger(EventLogger):
     def __init__(
         self,
@@ -118,6 +132,9 @@ def emit(self, event: Event) -> None:
         self._event_logger.emit(event)
 
 
+@deprecated(
+    "You should use `LoggerProvider` instead. Deprecated since version 1.35.0."
+)
 class EventLoggerProvider(ABC):
     @abstractmethod
     def get_event_logger(
@@ -130,6 +147,9 @@ def get_event_logger(
         """Returns an EventLoggerProvider for use."""
 
 
+@deprecated(
+    "You should use `NoOpLoggerProvider` instead. Deprecated since version 1.35.0."
+)
 class NoOpEventLoggerProvider(EventLoggerProvider):
     def get_event_logger(
         self,
@@ -143,6 +163,9 @@ def get_event_logger(
         )
 
 
+@deprecated(
+    "You should use `ProxyLoggerProvider` instead. Deprecated since version 1.35.0."
+)
 class ProxyEventLoggerProvider(EventLoggerProvider):
     def get_event_logger(
         self,

opentelemetry-sdk/src/opentelemetry/sdk/_events/__init__.py

@@ -15,6 +15,8 @@
 from time import time_ns
 from typing import Optional
 
+from typing_extensions import deprecated
+
 from opentelemetry import trace
 from opentelemetry._events import Event
 from opentelemetry._events import EventLogger as APIEventLogger
@@ -26,6 +28,9 @@
 _logger = logging.getLogger(__name__)
 
 
+@deprecated(
+    "You should use `Logger` instead. Deprecated since version 1.35.0."
+)
 class EventLogger(APIEventLogger):
     def __init__(
         self,
@@ -65,6 +70,9 @@ def emit(self, event: Event) -> None:
         self._logger.emit(log_record)
 
 
+@deprecated(
+    "You should use `LoggerProvider` instead. Deprecated since version 1.35.0."
+)
 class EventLoggerProvider(APIEventLoggerProvider):
     def __init__(self, logger_provider: Optional[LoggerProvider] = None):
         self._logger_provider = logger_provider or get_logger_provider()