fix(monitoring): don't override user set environment variables

Author: judahrand

src/bentoml/_internal/monitoring/otlp.py

@@ -4,7 +4,6 @@
 import datetime
 import logging
 import logging.config
-import os
 import random
 import typing as t
 
@@ -13,12 +12,6 @@
 from opentelemetry.sdk._logs import LoggerProvider
 from opentelemetry.sdk._logs import LoggingHandler
 from opentelemetry.sdk._logs.export import BatchLogRecordProcessor
-from opentelemetry.sdk.environment_variables import OTEL_EXPORTER_OTLP_CERTIFICATE
-from opentelemetry.sdk.environment_variables import OTEL_EXPORTER_OTLP_COMPRESSION
-from opentelemetry.sdk.environment_variables import OTEL_EXPORTER_OTLP_ENDPOINT
-from opentelemetry.sdk.environment_variables import OTEL_EXPORTER_OTLP_HEADERS
-from opentelemetry.sdk.environment_variables import OTEL_EXPORTER_OTLP_INSECURE
-from opentelemetry.sdk.environment_variables import OTEL_EXPORTER_OTLP_TIMEOUT
 from opentelemetry.sdk.resources import Resource
 
 from ...exceptions import MissingDependencyException
@@ -165,19 +158,6 @@ def _init_logger(self) -> None:
         )
         set_logger_provider(self.logger_provider)
 
-        if self.endpoint is not None:
-            os.environ[OTEL_EXPORTER_OTLP_ENDPOINT] = self.endpoint
-        if self.insecure is not None:
-            os.environ[OTEL_EXPORTER_OTLP_INSECURE] = str(self.insecure)
-        if self.credentials is not None:
-            os.environ[OTEL_EXPORTER_OTLP_CERTIFICATE] = self.credentials
-        if self.headers is not None:
-            os.environ[OTEL_EXPORTER_OTLP_HEADERS] = self.headers
-        if self.timeout is not None:
-            os.environ[OTEL_EXPORTER_OTLP_TIMEOUT] = str(self.timeout)
-        if self.compression is not None:
-            os.environ[OTEL_EXPORTER_OTLP_COMPRESSION] = self.compression
-
         exporter: OTLPHttpLogExporter | OTLPGrpcLogExporter
         if self.protocol == "http":
             exporter = OTLPHttpLogExporter(