Skip to content

Add information about 2 additional metrics#1705

Draft
mhenc wants to merge 2 commits intogoogle:mainfrom
mhenc:metrics_google_lib
Draft

Add information about 2 additional metrics#1705
mhenc wants to merge 2 commits intogoogle:mainfrom
mhenc:metrics_google_lib

Conversation

@mhenc
Copy link
Copy Markdown
Contributor

@mhenc mhenc commented Apr 29, 2026

Updated docs/observability/metrics.md to document the new native emission of Google GenAI metrics by ADK and the behavior when external instrumentation is present.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 29, 2026

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit 4ba3ff5
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/69f39fc9e94db1000892f181
😎 Deploy Preview https://deploy-preview-1705--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@mhenc mhenc force-pushed the metrics_google_lib branch from 858587e to a68a10a Compare April 30, 2026 09:38
Copy link
Copy Markdown
Collaborator

@koverholt koverholt left a comment

Choose a reason for hiding this comment

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

Thanks for the updates. I now see that #1686 is in a version of ADK Python that is not yet released, which was not clear in the original PR. We will need to un-publish that page and hold off on publishing it until a version of ADK is released with metrics support.

@koverholt
Copy link
Copy Markdown
Collaborator

Update: I opened #1713 to readd the metrics page when it's available in a release. Should the content in this PR be merged along with #1713, or is it dependent on other upstream changes?

From what I can tell, these two metrics are emitted by opentelemetry-instrumentation-google-genai, not by ADK itself.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants