Skip to content

[opentelemetry-instrumentation-genai-agno] Add instrumentation for agno model calls - #773

Open
DylanRussell wants to merge 1 commit into
DylanRussell/more_agno_instrumentation_4from
DylanRussell/more_agno_instrumentation_5
Open

DylanRussell wants to merge 1 commit into
DylanRussell/more_agno_instrumentation_4from
DylanRussell/more_agno_instrumentation_5

Conversation

@DylanRussell

Copy link
Copy Markdown
Contributor

Description

Probably want to wait until #765 lands to avoid duplicate inference calls, but eventually we will want this and that PR will not change the implementation here.

I think once this lands we will have reached parity with the agno instrumentations in open inference and other places..

Type of change

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Unot tests

Checklist

  • Followed the style guidelines of this project
  • Changelog updated if the change requires an entry
  • Unit tests added
  • Documentation updated

@DylanRussell
DylanRussell requested a review from a team as a code owner September 22, 2026 14:43
@DylanRussell DylanRussell changed the title [] Add instrumentation for agno model calls [opentelemetry-instrumentation-genai-agno] Add instrumentation for agno model calls Sep 22, 2026
@DylanRussell
DylanRussell added this pull request to stack #732 September 22, 2026 14:44
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Waiting on reviewers · refreshed 2026-10-17 01:44 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.

Copilot review overview

🟡 Changes recommended

Model spans start after inference, stream managers are wrapped incorrectly, and required tests are missing.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 2 High severity · 1 Medium severity · 3 Low severity

Open (6)
What changed in this PR

Adds Agno model-call telemetry for synchronous, asynchronous, streaming, multimodal, and error paths.

Changes:

  • Adds GenAI chat instrumentation and stream wrappers.
  • Adds provider/message/media extraction utilities.
  • Adds model tests and documentation.
File Description
patch.py Patches model response processing.
stream.py Adds model stream telemetry wrappers.
utils.py Adds model and multimodal extraction.
test_model.py Tests model instrumentation.
test_embedder.py Makes Gemini tests optional.
README.rst Documents model operations.
.changelog/​773.added Records the feature.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread instrumentation/opentelemetry-instrumentation-genai-agno/README.rst
@linux-foundation-easycla

linux-foundation-easycla Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

@DylanRussell
DylanRussell force-pushed the DylanRussell/more_agno_instrumentation_5 branch from 24e7603 to 2c94b98 Compare September 22, 2026 18:32
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment.
If you're still working on this, please add a comment or push new commits.

@github-actions github-actions Bot added the Stale Issue or PR has been inactive label Oct 8, 2026
DylanRussell added a commit that referenced this pull request Oct 13, 2026
@DylanRussell
DylanRussell force-pushed the DylanRussell/more_agno_instrumentation_5 branch from 7b28898 to 3165b6f Compare October 13, 2026 14:05
@github-actions github-actions Bot removed the Stale Issue or PR has been inactive label Oct 14, 2026
DylanRussell added a commit that referenced this pull request Oct 14, 2026
@DylanRussell
DylanRussell force-pushed the DylanRussell/more_agno_instrumentation_5 branch from 3165b6f to dcd39d4 Compare October 14, 2026 15:01
@DylanRussell
DylanRussell force-pushed the DylanRussell/more_agno_instrumentation_5 branch from dcd39d4 to 76089fc Compare October 16, 2026 20:31

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

Development

Successfully merging this pull request may close these issues.

2 participants