You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
... but those larger scope changes should be independent of the small incremental act of updating the Semantic Conventions to reflect the reality of instrumentation.
I've already drafted a small change to add generate_content as a documented value:
Area(s)
area:gen-ai
What's missing?
Instrumentation library for
google-genaihere:https://github.com/open-telemetry/opentelemetry-python-contrib/tree/6bde73ce344af1df7002bc001ad0fc7d51b4b577/instrumentation-genai/opentelemetry-instrumentation-google-genai
... adds
generate_contentas a value forgen_ai.operation.name:https://github.com/open-telemetry/opentelemetry-python-contrib/blob/6bde73ce344af1df7002bc001ad0fc7d51b4b577/instrumentation-genai/opentelemetry-instrumentation-google-genai/src/opentelemetry/instrumentation/google_genai/generate_content.py#L52
https://github.com/open-telemetry/opentelemetry-python-contrib/blob/6bde73ce344af1df7002bc001ad0fc7d51b4b577/instrumentation-genai/opentelemetry-instrumentation-google-genai/src/opentelemetry/instrumentation/google_genai/generate_content.py#L229
This should be captured in Semantic Conventions.
Describe the solution you'd like
For now, simply define
generate_contentas an additional permissible value.Separately, there are issues covering revisiting the meaning/scope in general of this operation name:
gen_ai.operation.nameto be defined in a way that allows for looser naming semantic-conventions-genai#21gen_ai.operation.namesemantic-conventions-genai#20... but those larger scope changes should be independent of the small incremental act of updating the Semantic Conventions to reflect the reality of instrumentation.
I've already drafted a small change to add
generate_contentas a documented value:I'm creating this issue as a point of reference for the CHANGELOG to go with the draft change. I intend to send a pull request soon.