Move environment variable to proper module

Author: xrmx

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

@@ -26,6 +26,9 @@
     _load_distro,
     _load_instrumentors,
 )
+from opentelemetry.instrumentation.environment_variables import (
+    OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH,
+)
 from opentelemetry.instrumentation.utils import _python_path_without_directory
 from opentelemetry.instrumentation.version import __version__
 from opentelemetry.util._importlib_metadata import entry_points
@@ -134,14 +137,14 @@ def initialize(*, swallow_exceptions: bool = True) -> None:
 
     # handle optional gevent monkey patching. This is done via environment variables so it may be used from the
     # opentelemetry operator
-    gevent_patch_env_variable_name = (
-        "OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH"
+    gevent_patch: str | None = environ.get(
+        OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH
     )
-    gevent_patch: str | None = environ.get(gevent_patch_env_variable_name)
     if gevent_patch is not None:
         if gevent_patch != "patch_all":
             _logger.error(
-                "%s values must be `patch_all`", gevent_patch_env_variable_name
+                "%s value must be `patch_all`",
+                OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH,
             )
         else:
             try:

opentelemetry-instrumentation/src/opentelemetry/instrumentation/environment_variables.py

@@ -26,3 +26,10 @@
 """
 .. envvar:: OTEL_PYTHON_CONFIGURATOR
 """
+
+OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH = (
+    "OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH"
+)
+"""
+.. envvar:: OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH
+"""

opentelemetry-instrumentation/tests/auto_instrumentation/test_initialize.py

@@ -84,7 +84,7 @@ def test_handles_invalid_gevent_monkeypatch(self, logger_mock):
         # pylint:disable=no-self-use
         auto_instrumentation.initialize()
         logger_mock.error.assert_called_once_with(
-            "%s values must be `patch_all`",
+            "%s value must be `patch_all`",
             "OTEL_PYTHON_AUTO_INSTRUMENTATION_EXPERIMENTAL_GEVENT_PATCH",
         )