Refactoring to remove duplicate codes

Author: shuningc

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

@@ -88,14 +88,21 @@ def _get_embedding_common_attributes(
     }
 
 
+def _get_span_name(
+    invocation: LLMInvocation | EmbeddingInvocation,
+) -> str:
+    """Get the span name for a GenAI invocation."""
+    return f"{invocation.operation_name} {invocation.request_model}".strip()
+
+
 def _get_llm_span_name(invocation: LLMInvocation) -> str:
     """Get the span name for an LLM invocation."""
-    return f"{invocation.operation_name} {invocation.request_model}".strip()
+    return _get_span_name(invocation)
 
 
 def _get_embedding_span_name(invocation: EmbeddingInvocation) -> str:
     """Get the span name for an Embedding invocation."""
-    return f"{invocation.operation_name} {invocation.request_model}".strip()
+    return _get_span_name(invocation)
 
 
 def _get_llm_messages_attributes_for_span(

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

@@ -201,6 +201,10 @@ class GenAIInvocation:
     context_token: ContextToken | None = None
     span: Span | None = None
     attributes: dict[str, Any] = field(default_factory=_new_str_any_dict)
+    # Monotonic start time in seconds (from timeit.default_timer) used
+    # for duration calculations to avoid mixing clock sources. This is
+    # populated by the TelemetryHandler when starting an invocation.
+    monotonic_start_s: float | None = None
 
 
 @dataclass
@@ -250,10 +254,6 @@ class LLMInvocation(GenAIInvocation):
     seed: int | None = None
     server_address: str | None = None
     server_port: int | None = None
-    # Monotonic start time in seconds (from timeit.default_timer) used
-    # for duration calculations to avoid mixing clock sources. This is
-    # populated by the TelemetryHandler when starting an invocation.
-    monotonic_start_s: float | None = None
 
 
 @dataclass
@@ -267,7 +267,6 @@ class EmbeddingInvocation(GenAIInvocation):
     operation_name: str = GenAI.GenAiOperationNameValues.EMBEDDINGS.value
 
     provider: str | None = None  # e.g., azure.ai.openai, openai, aws.bedrock
-
     request_model: str | None = None
     server_address: str | None = None
     server_port: int | None = None
@@ -291,7 +290,6 @@ class EmbeddingInvocation(GenAIInvocation):
     Additional attributes to set on metrics. Must be of a low cardinality.
     These attributes will not be set on spans or events.
     """
-    monotonic_start_s: float | None = None
 
 
 @dataclass