Default env var to NO_CONTENT when invalid envvar

Author: DylanRussell

util/opentelemetry-util-genai/src/opentelemetry/util/genai/utils.py

@@ -12,6 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+import logging
 import os
 
 from opentelemetry.instrumentation._semconv import (
@@ -24,6 +25,8 @@
 )
 from opentelemetry.util.genai.types import ContentCapturingMode
 
+logger = logging.getLogger(__name__)
+
 
 def get_content_capturing_mode() -> ContentCapturingMode:
     envvar = os.environ.get(OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT)
@@ -41,6 +44,7 @@ def get_content_capturing_mode() -> ContentCapturingMode:
     try:
         return ContentCapturingMode[envvar.upper()]
     except KeyError:
-        raise RuntimeError(
-            f"{envvar} is not a valid option for `OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT` environment variable. Must be one of {', '.join(e.name for e in ContentCapturingMode)}"
+        logger.warning(
+            f"{envvar} is not a valid option for `OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT` environment variable. Must be one of {', '.join(e.name for e in ContentCapturingMode)}. Defaulting to `NO_COTENT`"
         )
+        return ContentCapturingMode.NO_CONTENT

util/opentelemetry-util-genai/tests/test_utils.py

@@ -24,10 +24,10 @@
     OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT,
 )
 from opentelemetry.util.genai.types import ContentCapturingMode
-from opentelemetry.util.genai.utils import get_content_capturing_mode
+from opentelemetry.util.genai.utils import get_content_capturing_mode, logger
 
 
-def patch_env_vars(stability_mode, content_capturing):
+def patch_env_vars(*, stability_mode, content_capturing):
     def decorator(test_case):
         @patch.dict(
             os.environ,
@@ -49,7 +49,7 @@ def wrapper(*args, **kwargs):
 
 class TestVersion(unittest.TestCase):
     @patch_env_vars("gen_ai_latest_experimental", "SPAN_ONLY")
-    def test_get_content_caputring_mode_parses_valid_envvar(self):  # pylint: disable=no-self-use
+    def test_get_content_capturing_mode_parses_valid_envvar(self):  # pylint: disable=no-self-use
         _OpenTelemetrySemanticConventionStability._initialized = False
         assert get_content_capturing_mode() == ContentCapturingMode.SPAN_ONLY
 
@@ -70,5 +70,9 @@ def test_get_content_caputring_mode_raises_exception_when_semconv_stability_defa
     def test_get_content_caputring_mode_raises_exception_on_invalid_envvar(
         self,
     ):  # pylint: disable=no-self-use
-        with self.assertRaises(RuntimeError):
-            get_content_capturing_mode()
+        with self.assertLogs(logger, level="WARNING") as cm:
+            assert (
+                get_content_capturing_mode() == ContentCapturingMode.NO_CONTENT
+            )
+        self.assertEqual(len(cm.output), 1)
+        self.assertIn("INVALID_VALUE is not a valid option for ", cm.output[0])