Skip to content

Classify gen_ai.invocation.internal spans - #322

Open
RKest wants to merge 1 commit into
open-telemetry:mainfrom
RKest:genai-invocation-internal
Open

RKest wants to merge 1 commit into
open-telemetry:mainfrom
RKest:genai-invocation-internal

Conversation

@RKest

@RKest RKest commented Oct 16, 2026

Copy link
Copy Markdown

semantic-conventions-genai#571 replaces gen_ai.invoke_agent.internal and gen_ai.invoke_workflow.internal with gen_ai.invocation.internal, and adds the invoke_runner operation (ADK and OpenAI Agents Runner.run).

This maps invoke_agent, invoke_workflow, and invoke_runner to gen_ai.invocation.internal. The old types stay, so the currently pinned registry classifies as before: span kind already keeps only the types the pinned model declares.

semantic-conventions-genai#571 replaces gen_ai.invoke_agent.internal and gen_ai.invoke_workflow.internal with gen_ai.invocation.internal and adds the invoke_runner operation. Keep the old types so the current registry still classifies; span kind keeps only the types the pinned model declares.
@RKest
RKest requested a review from a team as a code owner October 16, 2026 18:12
Copilot AI balanced review requested due to automatic review settings October 16, 2026 18:12
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Waiting on reviewers · refreshed 2026-10-17 01:49 UTC

Review the latest changes.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused mappings preserve registry compatibility, and no blocking issues were identified.

0 open findings

What changed in this PR

Adds support for gen_ai.invocation.internal in GenAI span classification while retaining legacy mappings for the pinned registry.

Changes:

  • Maps agent, workflow, and runner operations to the unified internal span type.
  • Adds identifying attributes for unified invocations.
  • Tests unified classification and preserves client-agent classification.
File Description
tools/​gen-ai/​runner/​tests/​test_genai_coverage.py Tests unified invocation classification.
tools/​gen-ai/​runner/​src/​genai_conformance/​_coverage.py Adds unified operation and attribute mappings alongside legacy types.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch has not been deployed

No deployments
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.

2 participants