Update operation name to use a constant for consistency.

Author: michaelsafyan

instrumentation-genai/opentelemetry-instrumentation-google-genai/src/opentelemetry/instrumentation/google_genai/generate_content.py

@@ -45,6 +45,9 @@
 _logger = logging.getLogger(__name__)
 
 
+# Constant used for the value of 'gen_ai.operation.name".
+_GENERATE_CONTENT_OP_NAME = "generate_content"
+
 # Constant used to make the absence of content more understandable.
 _CONTENT_ELIDED = "<elided>"
 
@@ -220,13 +223,13 @@ def __init__(
 
     def start_span_as_current_span(self, model_name, function_name):
         return self._otel_wrapper.start_as_current_span(
-            f"generate_content {model_name}",
+            f"{_GENERATE_CONTENT_OP_NAME} {model_name}",
             start_time=self._start_time,
             attributes={
                 code_attributes.CODE_FUNCTION_NAME: function_name,
                 gen_ai_attributes.GEN_AI_SYSTEM: self._genai_system,
                 gen_ai_attributes.GEN_AI_REQUEST_MODEL: self._genai_request_model,
-                gen_ai_attributes.GEN_AI_OPERATION_NAME: "GenerateContent",
+                gen_ai_attributes.GEN_AI_OPERATION_NAME: _GENERATE_CONTENT_OP_NAME,
             },
         )
 
@@ -464,7 +467,7 @@ def _record_token_usage_metric(self):
                 gen_ai_attributes.GEN_AI_TOKEN_TYPE: "input",
                 gen_ai_attributes.GEN_AI_SYSTEM: self._genai_system,
                 gen_ai_attributes.GEN_AI_REQUEST_MODEL: self._genai_request_model,
-                gen_ai_attributes.GEN_AI_OPERATION_NAME: "GenerateContent",
+                gen_ai_attributes.GEN_AI_OPERATION_NAME: _GENERATE_CONTENT_OP_NAME,
             },
         )
         self._otel_wrapper.token_usage_metric.record(
@@ -473,15 +476,15 @@ def _record_token_usage_metric(self):
                 gen_ai_attributes.GEN_AI_TOKEN_TYPE: "output",
                 gen_ai_attributes.GEN_AI_SYSTEM: self._genai_system,
                 gen_ai_attributes.GEN_AI_REQUEST_MODEL: self._genai_request_model,
-                gen_ai_attributes.GEN_AI_OPERATION_NAME: "GenerateContent",
+                gen_ai_attributes.GEN_AI_OPERATION_NAME: _GENERATE_CONTENT_OP_NAME,
             },
         )
 
     def _record_duration_metric(self):
         attributes = {
             gen_ai_attributes.GEN_AI_SYSTEM: self._genai_system,
             gen_ai_attributes.GEN_AI_REQUEST_MODEL: self._genai_request_model,
-            gen_ai_attributes.GEN_AI_OPERATION_NAME: "GenerateContent",
+            gen_ai_attributes.GEN_AI_OPERATION_NAME: _GENERATE_CONTENT_OP_NAME,
         }
         if self._error_type is not None:
             attributes[error_attributes.ERROR_TYPE] = self._error_type

instrumentation-genai/opentelemetry-instrumentation-google-genai/tests/generate_content/nonstreaming_base.py

@@ -105,7 +105,7 @@ def test_generated_span_has_minimal_genai_attributes(self):
         span = self.otel.get_span_named("generate_content gemini-2.0-flash")
         self.assertEqual(span.attributes["gen_ai.system"], "gemini")
         self.assertEqual(
-            span.attributes["gen_ai.operation.name"], "GenerateContent"
+            span.attributes["gen_ai.operation.name"], "generate_content"
         )
 
     def test_generated_span_has_correct_function_name(self):
@@ -129,7 +129,7 @@ def test_generated_span_has_vertex_ai_system_when_configured(self):
         span = self.otel.get_span_named("generate_content gemini-2.0-flash")
         self.assertEqual(span.attributes["gen_ai.system"], "vertex_ai")
         self.assertEqual(
-            span.attributes["gen_ai.operation.name"], "GenerateContent"
+            span.attributes["gen_ai.operation.name"], "generate_content"
         )
 
     def test_generated_span_counts_tokens(self):