Fix RecursionError by preventing duplicate instrumentation in urllib3

Added a guard (_instrumented flag) to prevent urllib3 instrumentation from running multiple times, which caused RecursionError (#4532). This ensures _instrument() exits early if already applied.

Author: shrxya07

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

@@ -199,7 +199,7 @@ class RequestInfo:
 class URLLib3Instrumentor(BaseInstrumentor):
     def instrumentation_dependencies(self) -> Collection[str]:
         return _instruments
-
+    _instrumented = False
     def _instrument(self, **kwargs):
         """Instruments the urllib3 module
 
@@ -214,6 +214,9 @@ def _instrument(self, **kwargs):
                     list of regexes used to exclude URLs from tracking
         """
         # initialize semantic conventions opt-in if needed
+        if self._instrumented:
+            return
+        self._instrumented = True
         _OpenTelemetrySemanticConventionStability._initialize()
         sem_conv_opt_in_mode = _OpenTelemetrySemanticConventionStability._get_opentelemetry_stability_opt_in_mode(
             _OpenTelemetryStabilitySignalType.HTTP,