lint/refactor

Author: JWinermaSplunk

instrumentation/opentelemetry-instrumentation-logging/src/opentelemetry/instrumentation/logging/__init__.py

@@ -247,14 +247,12 @@ def record_factory(*args, **kwargs):
                 .lower()
                 == "true",
             )
-            handler_level = kwargs.get("log_handler_level", None)
-            if (
-                handler_level is None
-                and OTEL_PYTHON_LOG_HANDLER_LEVEL in environ
-            ):
-                handler_level = _get_log_level(
-                    environ[OTEL_PYTHON_LOG_HANDLER_LEVEL]
-                )
+            handler_level = kwargs.get(
+                "log_handler_level",
+                _get_log_level(environ[OTEL_PYTHON_LOG_HANDLER_LEVEL])
+                if OTEL_PYTHON_LOG_HANDLER_LEVEL in environ
+                else None,
+            )
             logger_provider = get_logger_provider()
             handler = _setup_logging_handler(
                 logger_provider=logger_provider,

instrumentation/opentelemetry-instrumentation-logging/src/opentelemetry/instrumentation/logging/handler.py

@@ -57,12 +57,10 @@ def _setup_logging_handler(
     level: int | None = None,
 ) -> LoggingHandler:
     handler = LoggingHandler(
-        level=logging.NOTSET,
+        level=level or logging.NOTSET,
         logger_provider=logger_provider,
         log_code_attributes=log_code_attributes,
     )
-    if level is not None:
-        handler.setLevel(level)
     logging.getLogger().addHandler(handler)
     _overwrite_logging_config_fns(handler)
     return handler