update naming of internal metric environment variable

Author: herin049

exporter/opentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py

@@ -103,7 +103,7 @@
     OTEL_EXPORTER_OTLP_HEADERS,
     OTEL_EXPORTER_OTLP_INSECURE,
     OTEL_EXPORTER_OTLP_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.environment_variables._internal import (
     parse_boolean_environment_variable,
@@ -397,7 +397,7 @@ def __init__(
             parsed_url,
             meter_provider,
             disabled=not parse_boolean_environment_variable(
-                OTEL_PYTHON_SDK_METRICS_ENABLED
+                OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
             ),
         )
 

exporter/opentelemetry-exporter-otlp-proto-grpc/tests/test_otlp_exporter_mixin.py

@@ -54,7 +54,7 @@
 from opentelemetry.sdk.environment_variables import (
     _OTEL_PYTHON_EXPORTER_OTLP_GRPC_CREDENTIAL_PROVIDER,
     OTEL_EXPORTER_OTLP_COMPRESSION,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.metrics import MeterProvider
 from opentelemetry.sdk.metrics.export import InMemoryMetricReader
@@ -388,7 +388,9 @@ def test_otlp_exporter_otlp_compression_envvar(
             ),
         )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     def test_shutdown(self):
         add_TraceServiceServicer_to_server(
             TraceServiceServicerWithExportParams(StatusCode.OK),
@@ -485,7 +487,9 @@ def test_export_over_closed_grpc_channel(self):
         system() == "Windows",
         "For gRPC + windows there's some added delay in the RPCs which breaks the assertion over amount of time passed.",
     )
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     def test_retry_info_is_respected(self):
         mock_trace_service = TraceServiceServicerWithExportParams(
             StatusCode.UNAVAILABLE,
@@ -628,7 +632,9 @@ def test_otlp_headers_from_env(self):
             (),
         )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     def test_permanent_failure(self):
         exporter = OTLPSpanExporterForTesting(
             insecure=True, meter_provider=self.meter_provider

exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py

@@ -61,7 +61,7 @@
     OTEL_EXPORTER_OTLP_LOGS_HEADERS,
     OTEL_EXPORTER_OTLP_LOGS_TIMEOUT,
     OTEL_EXPORTER_OTLP_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.environment_variables._internal import (
     parse_boolean_environment_variable,
@@ -162,7 +162,7 @@ def __init__(
             urlparse(self._endpoint),
             meter_provider,
             disabled=not parse_boolean_environment_variable(
-                OTEL_PYTHON_SDK_METRICS_ENABLED
+                OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
             ),
         )
 

exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py

@@ -86,7 +86,7 @@
     OTEL_EXPORTER_OTLP_METRICS_HEADERS,
     OTEL_EXPORTER_OTLP_METRICS_TIMEOUT,
     OTEL_EXPORTER_OTLP_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.environment_variables._internal import (
     parse_boolean_environment_variable,
@@ -227,7 +227,7 @@ def __init__(
             urlparse(self._endpoint),
             meter_provider,
             disabled=not parse_boolean_environment_variable(
-                OTEL_PYTHON_SDK_METRICS_ENABLED
+                OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
             ),
         )
 
@@ -412,7 +412,7 @@ def set_meter_provider(self, meter_provider: MeterProvider) -> None:
             urlparse(self._endpoint),
             meter_provider,
             disabled=not parse_boolean_environment_variable(
-                OTEL_PYTHON_SDK_METRICS_ENABLED
+                OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
             ),
         )
 

exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py

@@ -57,7 +57,7 @@
     OTEL_EXPORTER_OTLP_TRACES_ENDPOINT,
     OTEL_EXPORTER_OTLP_TRACES_HEADERS,
     OTEL_EXPORTER_OTLP_TRACES_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.environment_variables._internal import (
     parse_boolean_environment_variable,
@@ -157,7 +157,7 @@ def __init__(
             urlparse(self._endpoint),
             meter_provider,
             disabled=not parse_boolean_environment_variable(
-                OTEL_PYTHON_SDK_METRICS_ENABLED
+                OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
             ),
         )
 

exporter/opentelemetry-exporter-otlp-proto-http/tests/metrics/test_otlp_metrics_exporter.py

@@ -69,7 +69,7 @@
     OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE,
     OTEL_EXPORTER_OTLP_METRICS_TIMEOUT,
     OTEL_EXPORTER_OTLP_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.metrics import (
     Counter,
@@ -335,7 +335,9 @@ def test_headers_parse_from_env(self):
                 ),
             )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_success(self, mock_post):
         resp = Response()
@@ -374,7 +376,9 @@ def test_success(self, mock_post):
             metrics[2].data.data_points[0].attributes
         )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_failure(self, mock_post):
         resp = Response()
@@ -1294,7 +1298,9 @@ def test_preferred_aggregation_override(self):
             exporter._preferred_aggregation[Histogram], histogram_aggregation
         )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_retry_timeout(self, mock_post):
         exporter = OTLPMetricExporter(

exporter/opentelemetry-exporter-otlp-proto-http/tests/test_proto_log_exporter.py

@@ -58,7 +58,7 @@
     OTEL_EXPORTER_OTLP_LOGS_HEADERS,
     OTEL_EXPORTER_OTLP_LOGS_TIMEOUT,
     OTEL_EXPORTER_OTLP_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.metrics import MeterProvider
 from opentelemetry.sdk.metrics.export import InMemoryMetricReader
@@ -468,7 +468,9 @@ def test_2xx_status_code(self, mock_otlp_metric_exporter):
             LogRecordExportResult.SUCCESS,
         )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_retry_timeout(self, mock_post):
         exporter = OTLPLogExporter(
@@ -557,7 +559,9 @@ def test_export_no_collector_available_retryable(self, mock_post):
                 warning.records[0].message,
             )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_export_no_collector_available(self, mock_post):
         exporter = OTLPLogExporter(

exporter/opentelemetry-exporter-otlp-proto-http/tests/test_proto_span_exporter.py

@@ -48,7 +48,7 @@
     OTEL_EXPORTER_OTLP_TRACES_ENDPOINT,
     OTEL_EXPORTER_OTLP_TRACES_HEADERS,
     OTEL_EXPORTER_OTLP_TRACES_TIMEOUT,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.metrics import MeterProvider
 from opentelemetry.sdk.metrics.export import InMemoryMetricReader
@@ -299,7 +299,9 @@ def test_exporter_metrics_disabled_by_default(self, _mock_export):
 
         self.assertIsNone(self.metric_reader.get_metrics_data())
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_retry_timeout(self, mock_post):
         exporter = OTLPSpanExporter(
@@ -388,7 +390,9 @@ def test_export_no_collector_available_retryable(self, mock_post):
                 warning.records[0].message,
             )
 
-    @patch.dict("os.environ", {OTEL_PYTHON_SDK_METRICS_ENABLED: "true"})
+    @patch.dict(
+        "os.environ", {OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED: "true"}
+    )
     @patch.object(Session, "post")
     def test_export_no_collector_available(self, mock_post):
         exporter = OTLPSpanExporter(

opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/_logger_metrics.py

@@ -14,7 +14,7 @@
 
 from opentelemetry import metrics as metrics_api
 from opentelemetry.sdk.environment_variables import (
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.environment_variables._internal import (
     parse_boolean_environment_variable,
@@ -29,7 +29,7 @@ def __init__(self, meter_provider: metrics_api.MeterProvider) -> None:
         meter = meter_provider.get_meter("opentelemetry-sdk")
         self._created_logs = create_otel_sdk_log_created(meter)
         self._disabled = not parse_boolean_environment_variable(
-            OTEL_PYTHON_SDK_METRICS_ENABLED
+            OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
         )
 
     def emit_log(self) -> None:

opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py

@@ -46,7 +46,7 @@
     OTEL_BLRP_MAX_EXPORT_BATCH_SIZE,
     OTEL_BLRP_MAX_QUEUE_SIZE,
     OTEL_BLRP_SCHEDULE_DELAY,
-    OTEL_PYTHON_SDK_METRICS_ENABLED,
+    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED,
 )
 from opentelemetry.sdk.environment_variables._internal import (
     parse_boolean_environment_variable,
@@ -195,7 +195,7 @@ def __init__(
             OtelComponentTypeValues.SIMPLE_LOG_PROCESSOR,
             meter_provider or get_meter_provider(),
             disabled=not parse_boolean_environment_variable(
-                OTEL_PYTHON_SDK_METRICS_ENABLED
+                OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
             ),
         )
 
@@ -312,7 +312,7 @@ def __init__(
                 meter_provider or get_meter_provider(),
                 capacity=max_queue_size,
                 disabled=not parse_boolean_environment_variable(
-                    OTEL_PYTHON_SDK_METRICS_ENABLED
+                    OTEL_PYTHON_SDK_INTERNAL_METRICS_ENABLED
                 ),
             ),
         )