Skip to content

OpenAI integration update #4612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 7 commits into from
Jul 29, 2025
Merged

OpenAI integration update #4612

merged 7 commits into from
Jul 29, 2025

Conversation

antonpirker
Copy link
Member

@antonpirker antonpirker commented Jul 23, 2025 •
edited
Loading

antonpirker and others added 3 commits July 21, 2025 16:09
@antonpirker
Update openai spans to be OTel compatible (#4563)
dca9f30 
Change the spans name, op, and attributes to the new OTel compatible
names.

We checked and none of our customers use those legacy span names, span
ops, or attribute names in alerts, dashboards, or saved queries so this
is save to change in a minor.
@antonpirker
Merge branch 'master' into antonpirker/openai-overhaul
c263e02
@antonpirker @sentrivana
Add support for OpenAi responses API (#4564)
f0b3d95 
Add support for the new preferred way of using OpenAI, the `/responses`
API (see
https://platform.openai.com/docs/guides/responses-vs-chat-completions)

Also makes sure the current span is captured when an error is raised.

---------

Co-authored-by: Ivana Kellyer <[email protected]>
@codecov Codecov
Copy link

codecov bot commented Jul 23, 2025 •
edited
Loading

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
23717 1 23716 2283
View the top 1 failed test(s) by shortest run time 
tests.profiler.test_continuous_profiler::test_continuous_profiler_manual_start_and_stop_sampled[experiment-start_profiler/stop_profiler-thread]
Stack Traces | 0.637s run time 
tests/profiler/test_continuous_profiler.py:349: in test_continuous_profiler_manual_start_and_stop_sampled
    assert_single_transaction_with_profile_chunks(envelopes, thread)
tests/profiler/test_continuous_profiler.py:140: in assert_single_transaction_with_profile_chunks
    assert len(items["profile_chunk"]) > 0
E   assert 0 > 0
E    +  where 0 = len([])

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

antonpirker and others added 4 commits July 25, 2025 09:24
@antonpirker
Merge branch 'master' into antonpirker/openai-overhaul
5cc512f
@antonpirker @sentrivana
Make OpenAI span data OTel compatible (#4613)
fbbb5c4 
Make sure all OpenTelemetry semantic conventions for AI clients and also
the Sentry Conventions are adhered to.

Also cleans up the code a bit for better readability/maintenance.

---------

Co-authored-by: Ivana Kellyer <[email protected]>
@antonpirker @sentrivana
OpenAI pipeline name and streaming attribute (#4628)
c39d47f 
Rename attributes for AI pipeline name and the streaming boolean.

---------

Co-authored-by: Ivana Kellyer <[email protected]>
@antonpirker
Merge branch 'master' into antonpirker/openai-overhaul
2b9cbfd
@antonpirker antonpirker marked this pull request as ready for review July 29, 2025 10:10
@antonpirker antonpirker requested a review from a team as a code owner July 29, 2025 10:10
@antonpirker antonpirker merged commit fcace85 into master Jul 29, 2025
144 of 147 checks passed
@antonpirker antonpirker deleted the antonpirker/openai-overhaul branch July 29, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sentrivana sentrivana sentrivana approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@antonpirker @sentrivana