Skip to content

Add support for workflow invocation in genAI utils#4334

Closed
wrisa wants to merge 20 commits into
open-telemetry:mainfrom
wrisa:workflow-handler-genai-utils
Closed

Add support for workflow invocation in genAI utils#4334
wrisa wants to merge 20 commits into
open-telemetry:mainfrom
wrisa:workflow-handler-genai-utils

Conversation

@wrisa
Copy link
Copy Markdown
Contributor

@wrisa wrisa commented Mar 13, 2026

The commits got messed up. Please review this PR instead.

@wrisa wrisa marked this pull request as ready for review March 18, 2026 18:20
@wrisa wrisa requested a review from a team as a code owner March 18, 2026 18:20
Copy link
Copy Markdown
Member

@lmolkova lmolkova left a comment

Choose a reason for hiding this comment

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

LGTM with a caveat on reusing common fail/stop methods being introduced in #4219

invocation.span.end()
return invocation

def fail_workflow( # pylint: disable=no-self-use
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

discussed on the SIG call, let's reuse common fail and stop methods

wrisa and others added 9 commits March 20, 2026 08:58
)

Co-authored-by: otelbot <197425009+otelbot@users.noreply.github.com>
…ssages (open-telemetry#4349)

Matches Java instrumentation behavior: only create a recv span when a
message is actually received, avoiding empty spans on idle polls.

Fixes open-telemetry#3590

Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>
…l deltas with arguments=None. (open-telemetry#4350)

* wip: fix for none/null in function tool calls.

* polish: add changelog.

* wip: fixing lint and precommit.

* wip: converted class based tests.

* wip: fixing the precommit.
@wrisa wrisa deleted the workflow-handler-genai-utils branch March 31, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

8 participants