Type hinting, re-raise original exception

Author: phillipuniverse

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

@@ -118,7 +118,7 @@ def run() -> None:
     execl(executable, executable, *args.command_args)
 
 
-def initialize(*, swallow_exceptions=True):
+def initialize(*, swallow_exceptions: bool = True):
     """
     Setup auto-instrumentation, called by the sitecustomize module
 
@@ -138,6 +138,4 @@ def initialize(*, swallow_exceptions=True):
     except Exception as exc:  # pylint: disable=broad-except
         _logger.exception("Failed to auto initialize OpenTelemetry")
         if not swallow_exceptions:
-            raise ValueError(
-                "Failed to auto initialize OpenTelemetry"
-            ) from exc
+            raise exc

opentelemetry-instrumentation/tests/auto_instrumentation/test_initialize.py

@@ -64,13 +64,11 @@ def test_handles_exceptions(self, load_distro_mock, logger_mock):
     @patch("opentelemetry.instrumentation.auto_instrumentation._load_distro")
     def test_reraises_exceptions(self, load_distro_mock, logger_mock):
         # pylint:disable=no-self-use
-        load_distro_mock.side_effect = ValueError
+        load_distro_mock.side_effect = ValueError("inner exception")
         with self.assertRaises(ValueError) as em:
             auto_instrumentation.initialize(swallow_exceptions=False)
             logger_mock.exception.assert_called_once_with(
                 "Failed to auto initialize OpenTelemetry"
             )
 
-        self.assertEqual(
-            "Failed to auto initialize OpenTelemetry", str(em.exception)
-        )
+        self.assertEqual("inner exception", str(em.exception))