Allow reraising the root exception if instrumentation fails

I would rather completely fail startup in my services if instrumentation fails for whatever reason instead of just logging an exception and continuing.

Use case:

from opentelemetry import autoinstrumentation

autoinstrumentation.initialize(reraise=True)

Author: phillipuniverse

CHANGELOG.md

@@ -37,6 +37,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   ([#3520](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3520))
 - `opentelemetry-instrumentation-botocore` Ensure spans end on early stream closure for Bedrock Streaming APIs
   ([#3481](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3481))
+- `opentelemetry-instrumentation` Allow re-raising exception when instrumentation fails
+  ([#3545](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3545))
 
 ### Breaking changes
 

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

@@ -118,8 +118,12 @@ def run() -> None:
     execl(executable, executable, *args.command_args)
 
 
-def initialize():
-    """Setup auto-instrumentation, called by the sitecustomize module"""
+def initialize(*, reraise=False):
+    """
+    Setup auto-instrumentation, called by the sitecustomize module
+
+    :param reraise: Whether or not to re-raise exceptions in the auto-instrumentation process to the caller. Exceptions are logged and swalloed by default.
+    """
     # prevents auto-instrumentation of subprocesses if code execs another python process
     if "PYTHONPATH" in environ:
         environ["PYTHONPATH"] = _python_path_without_directory(
@@ -131,5 +135,7 @@ def initialize():
         distro.configure()
         _load_configurators()
         _load_instrumentors(distro)
-    except Exception:  # pylint: disable=broad-except
+    except Exception as e:  # pylint: disable=broad-except
         _logger.exception("Failed to auto initialize OpenTelemetry")
+        if reraise:
+            raise ValueError("Failed to auto initialize OpenTelemetry") from e

opentelemetry-instrumentation/tests/auto_instrumentation/test_initialize.py

@@ -59,3 +59,14 @@ def test_handles_exceptions(self, load_distro_mock, logger_mock):
         logger_mock.exception.assert_called_once_with(
             "Failed to auto initialize OpenTelemetry"
         )
+
+    @patch("opentelemetry.instrumentation.auto_instrumentation._logger")
+    @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
+        with self.assertRaises(ValueError):
+            auto_instrumentation.initialize(reraise=True)
+            logger_mock.exception.assert_called_once_with(
+                "Failed to auto initialize OpenTelemetry"
+            )