Skip to content
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

Add support for OpenAI "Developer Messages" introduced in openAI java client 0.8 #532

Open
JonasKunz opened this issue Feb 5, 2025 · 1 comment · May be fixed by #539
Open

Add support for OpenAI "Developer Messages" introduced in openAI java client 0.8 #532

JonasKunz opened this issue Feb 5, 2025 · 1 comment · May be fixed by #539
Assignees
@JonasKunz
Labels
agent-java

Comments

@JonasKunz
Copy link
Contributor

JonasKunz commented Feb 5, 2025 •
edited
Loading

While implementing #531, I discoveres that openAI added developer messages as replacement for system messages.
We'll need to decide on how to handle them, because currently the semantic conventions don't cover those.
At the moment the instrumentation will fail if event logging is enabled, because it encounteres an unknown type.
Some possible alternatives:

  • Don't report as an event and wait for semconv
  • Report as system.message for now
@JonasKunz JonasKunz self-assigned this Feb 5, 2025
@JonasKunz
Copy link
Contributor Author

Related semconv issue: open-telemetry/semantic-conventions#1877

@JonasKunz JonasKunz linked a pull request Feb 11, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JonasKunz JonasKunz

Labels
agent-java
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix support for OpenAI developer messages JonasKunz/elastic-otel-java
1 participant
@JonasKunz