Skip to content

HYBIM-802 Updated utils APIs#348

Merged
wrisa merged 3 commits into
mainfrom
update-utils-APIs-to-match-upstream
Jun 25, 2026
Merged

HYBIM-802 Updated utils APIs#348
wrisa merged 3 commits into
mainfrom
update-utils-APIs-to-match-upstream

Conversation

@wrisa

@wrisa wrisa commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

To match new upstream APIs,
added
inference, embedding, tool, workflow

deprecated

start_inference
start_embedding
start_tool
start_workflow

Note:
Not deprecating start_agent and adding invoke_local_agent similar to upstream becaset hese are not a direct equivalent. The splunk-distro start_agent accepts AgentCreation | AgentInvocation (supports two types) and does more work (context stack, conversation root marking, etc.), while the upstream invoke_local_agent is specifically for AgentInvocation with SpanKind.INTERNAL.

@wrisa wrisa changed the title Updated utils APIs HYBIM-802 Updated utils APIs Jun 17, 2026
@wrisa wrisa marked this pull request as ready for review June 17, 2026 16:55
@wrisa wrisa requested review from a team as code owners June 17, 2026 16:55

@keith-decker keith-decker 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.

LGTM, only one non blocking note from codex:

The PR changes public API guidance by marking the start_* factory methods deprecated in favor of the shorter aliases, but the package changelog's [Unreleased] section does not mention that API transition. Prior handler API work in PR 293 updated this changelog, so adding a short Changed entry would make the release notes match the public surface change.

Comment thread util/opentelemetry-util-genai/src/opentelemetry/util/genai/handler.py Outdated
Comment thread util/opentelemetry-util-genai/src/opentelemetry/util/genai/handler.py Outdated
Comment thread util/opentelemetry-util-genai/src/opentelemetry/util/genai/handler.py Outdated
wrisa and others added 3 commits June 24, 2026 10:10
Wrap long line in test_embedding_context_manager to satisfy ruff format check.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wrisa wrisa force-pushed the update-utils-APIs-to-match-upstream branch from 695ba88 to c56d23c Compare June 24, 2026 17:10
@wrisa wrisa merged commit cfa13fb into main Jun 25, 2026
14 checks passed
@wrisa wrisa deleted the update-utils-APIs-to-match-upstream branch June 25, 2026 16:26
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants