Use OTEL_PYTHON_LOG_LEVEL instead

Author: jeremydvoss

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

@@ -47,7 +47,7 @@
     OTEL_EXPORTER_OTLP_TRACES_PROTOCOL,
     OTEL_TRACES_SAMPLER,
     OTEL_TRACES_SAMPLER_ARG,
-    OTEL_LOG_LEVEL,
+    OTEL_PYTHON_LOG_LEVEL,
     OTEL_PYTHON_LOG_FORMAT,
 )
 from opentelemetry.sdk.metrics import MeterProvider
@@ -91,7 +91,7 @@
 
 _OTEL_SAMPLER_ENTRY_POINT_GROUP = "opentelemetry_traces_sampler"
 
-_OTEL_LOG_LEVEL_BY_NAME = {
+_OTEL_PYTHON_LOG_LEVEL_BY_NAME = {
     "notset": logging.NOTSET,
     "debug": logging.DEBUG,
     "info": logging.INFO,
@@ -144,7 +144,7 @@ def _get_id_generator() -> str:
     return environ.get(OTEL_PYTHON_ID_GENERATOR, _DEFAULT_ID_GENERATOR)
 
 def _get_log_level() -> int:
-    return _OTEL_LOG_LEVEL_BY_NAME.get(environ.get(OTEL_LOG_LEVEL, "notset").lower().strip(), logging.NOTSET)
+    return _OTEL_PYTHON_LOG_LEVEL_BY_NAME.get(environ.get(OTEL_PYTHON_LOG_LEVEL, "notset").lower().strip(), logging.NOTSET)
 
 
 def _get_exporter_entry_point(
@@ -275,7 +275,7 @@ def _init_logging(
             logger_provider=provider
         )
         # Log level
-        if OTEL_LOG_LEVEL in environ:
+        if OTEL_PYTHON_LOG_LEVEL in environ:
             handler.setLevel(_get_log_level())
         # Log format
         if OTEL_PYTHON_LOG_FORMAT in environ:

opentelemetry-sdk/tests/test_configurator.py

@@ -687,7 +687,7 @@ def test_logging_init_exporter(self):
         environ,
         {
             "OTEL_RESOURCE_ATTRIBUTES": "service.name=otlp-service",
-            "OTEL_LOG_LEVEL": "CUSTOM_LOG_LEVEL",
+            "OTEL_PYTHON_LOG_LEVEL": "CUSTOM_LOG_LEVEL",
         },
         clear=True,
     )
@@ -724,7 +724,7 @@ def test_logging_init_exporter_level_under(self, log_level_mock):
         environ,
         {
             "OTEL_RESOURCE_ATTRIBUTES": "service.name=otlp-service",
-            "OTEL_LOG_LEVEL": "CUSTOM_LOG_LEVEL",
+            "OTEL_PYTHON_LOG_LEVEL": "CUSTOM_LOG_LEVEL",
         },
         clear=True,
     )
@@ -827,35 +827,35 @@ def test_logging_init_exporter_without_handler_setup(self):
                 self.fail()
 
     @patch.dict(environ, {}, clear=True)
-    def test_otel_log_level_by_name_default(self):
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_default(self):
         self.assertEqual(_get_log_level(), NOTSET)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": "NOTSET "}, clear=True)
-    def test_otel_log_level_by_name_notset(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": "NOTSET "}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_notset(self):
         self.assertEqual(_get_log_level(), NOTSET)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": " DeBug "}, clear=True)
-    def test_otel_log_level_by_name_debug(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": " DeBug "}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_debug(self):
         self.assertEqual(_get_log_level(), DEBUG)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": " info "}, clear=True)
-    def test_otel_log_level_by_name_info(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": " info "}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_info(self):
         self.assertEqual(_get_log_level(), INFO)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": " warn"}, clear=True)
-    def test_otel_log_level_by_name_warn(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": " warn"}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_warn(self):
         self.assertEqual(_get_log_level(), WARNING)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": " warnING "}, clear=True)
-    def test_otel_log_level_by_name_warning(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": " warnING "}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_warning(self):
         self.assertEqual(_get_log_level(), WARNING)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": " eRroR"}, clear=True)
-    def test_otel_log_level_by_name_error(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": " eRroR"}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_error(self):
         self.assertEqual(_get_log_level(), ERROR)
 
-    @patch.dict(environ, {"OTEL_LOG_LEVEL": "foobar"}, clear=True)
-    def test_otel_log_level_by_name_invalid(self):
+    @patch.dict(environ, {"OTEL_PYTHON_LOG_LEVEL": "foobar"}, clear=True)
+    def test_OTEL_PYTHON_LOG_LEVEL_by_name_invalid(self):
         self.assertEqual(_get_log_level(), NOTSET)
 
     @patch.dict(