search

open-telemetry / opentelemetry-python-contrib

OpenTelemetry instrumentation for Python modules
https://opentelemetry.io
Apache License 2.0
739 stars 612 forks source link

[release/v1.28.x-0.49bx] Add openai instrumentation to opentelemetry-bootstrap (#2996) #3010

Closed aabmass closed 6 days ago

aabmass commented 1 week ago

Clean cherry-pick of https://github.com/open-telemetry/opentelemetry-python-contrib/pull/2996 to the release/v1.28.x-0.49bx branch.

Part of https://github.com/open-telemetry/opentelemetry-python/issues/4281

aabmass commented 6 days ago

Misc 0 / docs (pull_request) Failing after 35s

Docs build is failing because of a limitation with our automation. I am going to ignore for this release branch since it is usually skipped in the cherry pick workflow run by opentelemetrybot

aabmass commented 6 days ago

@lzchen I had to run tox -e generate to fix CI as well 425b332126f40134eb34d48f32e02738e5161c2d. I think this is expected and I'm assuming the Prepare Release workflow will update it to the right version when I cut the patch release?

lzchen commented 6 days ago

@lzchen I had to run tox -e generate to fix CI as well 425b332. I think this is expected and I'm assuming the Prepare Release workflow will update it to the right version when I cut the patch release?

Since openai instrumentation is released via a different workflow, it actually doesn't matter what the gen script changes the version to. In this case, it's actually "behind" on this release branch since the 2.0b0 release of openai instrumentation was already released after this release branch was created. I think you can just generate the code just to get builds to pass and it should be fine.