Skip to content

Conversation

@dmontagu
Copy link
Contributor

Requested recently in the context of the openai instrumentation not following semantic conventions. I'll also open a PR on pydantic-ai to reflect updates to the semantic conventions that have happened since the last modification there.

@cloudflare-workers-and-pages
Copy link

Deploying logfire-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0d3ae75
Status: ✅  Deploy successful!
Preview URL: https://07edc372.logfire-docs.pages.dev
Branch Preview URL: https://update-llm-instrumentation-s.logfire-docs.pages.dev

View logs

@rolandgvc
Copy link

following, it would be great to get this in main! I'm looking forward to better standardization with OTEL

Copy link

@brightsparc brightsparc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great I can see that antropic and openai chat completions are using new format, but the openai responses endpoint seems to be using the previous events schema.

OPERATION_NAME: 'chat',
REQUEST_MODEL: json_data.get('model'),
'request_data': {'model': json_data.get('model'), 'stream': stream},
'events': inputs_to_events(

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you plan to also update this to be gen_ai.input.messages and gen_ai.output.messages

Copy link

@brightsparc brightsparc Dec 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dmontagu i've created a new PR #1590 from my repo to replace events with gen_ai.input.messages and gen_ai.output.messages. Could also modify this to keep both formats if required.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brightsparc @dmontagu Started PR #1593 to port in the rest of the Semantic Conventions for Anthropic and OpenAI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants