OpenAI v2 onboard onto semantic conventions 1.37.0: chat history and other breaking changes #3715
Conversation
github-actions
bot
assigned
codefromthecrypt,
karthikscale3,
lmolkova,
lzchen and
nirga
lmolkova
changed the title
|
As a common package we have https://github.com/open-telemetry/opentelemetry-python-contrib/tree/main/util/opentelemetry-util-genai |
|
thanks a lot, @xrmx! |
zhirafovod
left a comment
zhirafovod
left a comment
There was a problem hiding this comment.
Some minor suggestions, but overall it looks good for this iteration. Thanks for doing it, we can add some notes to follow up on these items in the following pr.
instrumentation-genai/opentelemetry-instrumentation-openai-v2/examples/manual/.env
Outdated
Show resolved
Hide resolved
...opentelemetry-instrumentation-openai-v2/src/opentelemetry/instrumentation/openai_v2/patch.py
Show resolved
Hide resolved
...ntelemetry-instrumentation-openai-v2/src/opentelemetry/instrumentation/openai_v2/__init__.py
Outdated
Show resolved
Hide resolved
@xrmx , I would propose we merge this one before the common library is ready. It will provide testing needed to switch to the new library. Also, please review the PRs which are pre-requirement to switching to the util library. |
lmolkova
force-pushed
the
openai-v2-input-output-attrs
branch
from
4d9ba77 to
4887a19
Compare
|
|
||
|
|
||
| def assert_log_parent(log, span): | ||
| """Assert that the log record has the correct parent span context""" |
| "opentelemetry-instrumentation ~= 0.58b0", | ||
| "opentelemetry-semantic-conventions ~= 0.58b0" | ||
| "opentelemetry-semantic-conventions ~= 0.58b0", | ||
| "opentelemetry-util-genai >= 0.2b0, <0.3b0", |
There was a problem hiding this comment.
TODO: 0.3b0.dev is required
lmolkova
force-pushed
the
openai-v2-input-output-attrs
branch
from
a1bbbf9 to
3f0e18a
Compare
8 participants
Implements new semconv 1.37.0 behind
OTEL_SEMCONV_STABILITY_OPT_IN = gen_ai_latest_experimentalopt-in.TODOs: