Move Advisory types to metrics _internal and export the public one

Author: xrmx

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

@@ -55,6 +55,8 @@
     Counter,
     Histogram,
     Instrument,
+    MetricsCommonAdvisory,
+    MetricsHistogramAdvisory,
     NoOpCounter,
     NoOpHistogram,
     NoOpObservableCounter,
@@ -129,4 +131,6 @@
     "Observation",
     "CallbackT",
     "NoOpMeter",
+    "MetricsCommonAdvisory",
+    "MetricsHistogramAdvisory",
 ]

opentelemetry-api/src/opentelemetry/metrics/_internal/__init__.py

@@ -54,6 +54,8 @@
     Counter,
     Gauge,
     Histogram,
+    MetricsCommonAdvisory,
+    MetricsHistogramAdvisory,
     NoOpCounter,
     NoOpGauge,
     NoOpHistogram,
@@ -65,6 +67,7 @@
     ObservableGauge,
     ObservableUpDownCounter,
     UpDownCounter,
+    _MetricsInstrumentAdvisory,
     _ProxyCounter,
     _ProxyGauge,
     _ProxyHistogram,
@@ -77,9 +80,6 @@
 from opentelemetry.util._providers import _load_provider
 from opentelemetry.util.types import (
     Attributes,
-    MetricsCommonAdvisory,
-    MetricsHistogramAdvisory,
-    MetricsInstrumentAdvisory,
 )
 
 _logger = getLogger(__name__)
@@ -188,7 +188,7 @@ class _InstrumentRegistrationStatus:
     instrument_id: str
     already_registered: bool
     conflict: bool
-    current_advisory: Optional[MetricsInstrumentAdvisory]
+    current_advisory: Optional[_MetricsInstrumentAdvisory]
 
 
 class Meter(ABC):
@@ -209,7 +209,7 @@ def __init__(
         self._version = version
         self._schema_url = schema_url
         self._instrument_ids: Dict[
-            str, Optional[MetricsInstrumentAdvisory]
+            str, Optional[_MetricsInstrumentAdvisory]
         ] = {}
         self._instrument_ids_lock = Lock()
 
@@ -240,7 +240,7 @@ def _register_instrument(
         type_: type,
         unit: str,
         description: str,
-        advisory: Optional[MetricsInstrumentAdvisory] = None,
+        advisory: Optional[_MetricsInstrumentAdvisory] = None,
     ) -> _InstrumentRegistrationStatus:
         """
         Register an instrument with the name, type, unit and description as

opentelemetry-api/src/opentelemetry/metrics/_internal/instrument.py

@@ -38,9 +38,6 @@
 from opentelemetry.metrics._internal.observation import Observation
 from opentelemetry.util.types import (
     Attributes,
-    MetricsCommonAdvisory,
-    MetricsHistogramAdvisory,
-    MetricsInstrumentAdvisory,
 )
 
 _logger = getLogger(__name__)
@@ -49,6 +46,21 @@
 _unit_regex = re_compile(r"[\x00-\x7F]{0,63}")
 
 
+@dataclass(frozen=True)
+class MetricsHistogramAdvisory:
+    explicit_bucket_boundaries: Optional[Sequence[float]] = None
+
+
+@dataclass(frozen=True)
+class MetricsCommonAdvisory:
+    pass
+
+
+_MetricsInstrumentAdvisory = Union[
+    MetricsCommonAdvisory, MetricsHistogramAdvisory
+]
+
+
 @dataclass(frozen=True)
 class CallbackOptions:
     """Options for the callback
@@ -78,7 +90,7 @@ def __init__(
         name: str,
         unit: str = "",
         description: str = "",
-        advisory: Optional[MetricsInstrumentAdvisory] = None,
+        advisory: Optional[_MetricsInstrumentAdvisory] = None,
     ) -> None:
         pass
 
@@ -124,7 +136,7 @@ def __init__(
         name: str,
         unit: str = "",
         description: str = "",
-        advisory: Optional[MetricsInstrumentAdvisory] = None,
+        advisory: Optional[_MetricsInstrumentAdvisory] = None,
     ) -> None:
         self._name = name
         self._unit = unit
@@ -152,7 +164,7 @@ def __init__(
         callbacks: Optional[Sequence[CallbackT]] = None,
         unit: str = "",
         description: str = "",
-        advisory: Optional[MetricsInstrumentAdvisory] = None,
+        advisory: Optional[_MetricsInstrumentAdvisory] = None,
     ) -> None:
         super().__init__(name, unit, description, advisory=advisory)
         self._callbacks = callbacks
@@ -172,7 +184,7 @@ def __init__(
         callbacks: Optional[Sequence[CallbackT]] = None,
         unit: str = "",
         description: str = "",
-        advisory: Optional[MetricsInstrumentAdvisory] = None,
+        advisory: Optional[_MetricsInstrumentAdvisory] = None,
     ) -> None:
         super().__init__(
             name, unit=unit, description=description, advisory=advisory

opentelemetry-api/src/opentelemetry/util/types.py

@@ -12,7 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-from dataclasses import dataclass
 from typing import Mapping, Optional, Sequence, Tuple, Union
 
 # This is the implementation of the "Any" type as specified by the specifications of OpenTelemetry data model for logs.
@@ -56,18 +55,3 @@
     ],
     ...,
 ]
-
-
-@dataclass
-class MetricsHistogramAdvisory:
-    explicit_bucket_boundaries: Optional[Sequence[float]] = None
-
-
-@dataclass
-class MetricsCommonAdvisory:
-    pass
-
-
-MetricsInstrumentAdvisory = Union[
-    MetricsCommonAdvisory, MetricsHistogramAdvisory
-]

opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/instrument.py

@@ -21,7 +21,7 @@
 # This kind of import is needed to avoid Sphinx errors.
 import opentelemetry.sdk.metrics
 from opentelemetry.context import Context, get_current
-from opentelemetry.metrics import CallbackT
+from opentelemetry.metrics import CallbackT, MetricsHistogramAdvisory
 from opentelemetry.metrics import Counter as APICounter
 from opentelemetry.metrics import Histogram as APIHistogram
 from opentelemetry.metrics import ObservableCounter as APIObservableCounter
@@ -34,7 +34,6 @@
 from opentelemetry.metrics._internal.instrument import CallbackOptions
 from opentelemetry.sdk.metrics._internal.measurement import Measurement
 from opentelemetry.sdk.util.instrumentation import InstrumentationScope
-from opentelemetry.util.types import MetricsInstrumentAdvisory
 
 _logger = getLogger(__name__)
 
@@ -227,7 +226,7 @@ def __init__(
         measurement_consumer: "opentelemetry.sdk.metrics.MeasurementConsumer",
         unit: str = "",
         description: str = "",
-        advisory: Optional[MetricsInstrumentAdvisory] = None,
+        advisory: Optional[MetricsHistogramAdvisory] = None,
     ):
         super().__init__(
             name,