Skip to content

Add information about 2 additional metrics#1705

Draft
mhenc wants to merge 3 commits into
google:mainfrom
mhenc:metrics_google_lib
Draft

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

Conversation

@mhenc

@mhenc mhenc commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

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

netlify Bot commented Apr 29, 2026

Copy link
Copy Markdown

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit 5e78804
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/69f909e47478b50008f360af
😎 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

@koverholt koverholt left a comment

Copy link
Copy Markdown
Collaborator

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.

@mhenc

mhenc commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

Sorry for not letting you know that metrics are not yet released.

No, this change still waits for the PR to be merged. I will let you know when it's ok to review this one.

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