Minor refactor of basic config patch

Author: pmcollins

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

@@ -251,41 +251,43 @@ def _init_logging(
     set_event_logger_provider(event_logger_provider)
 
     if setup_logging_handler:
-        # noinspection PyPep8Naming
-        original_basicConfig = logging.basicConfig
+        _patch_basic_config()
 
-        # noinspection PyPep8Naming
-        def patched_basicConfig(*args, **kwargs):
-            # Get the root logger
-            root = logging.getLogger()
+        # Add OTel handler
+        handler = LoggingHandler(
+            level=logging.NOTSET, logger_provider=provider
+        )
+        logging.getLogger().addHandler(handler)
 
-            # Check if the only handler is our OTel handler
-            has_only_otel = len(root.handlers) == 1 and isinstance(
-                root.handlers[0], LoggingHandler
-            )
 
-            if has_only_otel:
-                # Temporarily remove OTel handler
-                otel_handler = root.handlers[0]
-                root.handlers = []
+def _patch_basic_config():
+    original_basic_config = logging.basicConfig
 
-                # Call original basicConfig
-                original_basicConfig(*args, **kwargs)
+    def patched_basic_config(*args, **kwargs):
+        # Get the root logger
+        root = logging.getLogger()
 
-                # Add OTel handler back
-                root.addHandler(otel_handler)
-            else:
-                # Normal behavior
-                original_basicConfig(*args, **kwargs)
+        # Check if the only handler is our OTel handler
+        has_only_otel = len(root.handlers) == 1 and isinstance(
+            root.handlers[0], LoggingHandler
+        )
 
-        # Apply the monkey patch
-        logging.basicConfig = patched_basicConfig
+        if has_only_otel:
+            # Temporarily remove OTel handler
+            otel_handler = root.handlers[0]
+            root.handlers = []
 
-        # Add OTel handler
-        handler = LoggingHandler(
-            level=logging.NOTSET, logger_provider=provider
-        )
-        logging.getLogger().addHandler(handler)
+            # Call original basicConfig
+            original_basic_config(*args, **kwargs)
+
+            # Add OTel handler back
+            root.addHandler(otel_handler)
+        else:
+            # Normal behavior
+            original_basic_config(*args, **kwargs)
+
+    # Apply the monkey patch
+    logging.basicConfig = patched_basic_config
 
 
 def _import_exporters(