Add comprehensive OpenTelemetry tracing documentation for Agent Engine#1537
Open
perashanid wants to merge 1 commit intogoogle:mainfrom
Open
Add comprehensive OpenTelemetry tracing documentation for Agent Engine#1537perashanid wants to merge 1 commit intogoogle:mainfrom
perashanid wants to merge 1 commit intogoogle:mainfrom
Conversation
- Add detailed guide for implementing OTEL tracing with Agent Engine - Document critical set_up() initialization requirement - Include architecture diagrams and data flow explanations - Provide both environment variable and manual configuration methods - Add infrastructure setup examples (Terraform, IAM) - Include comprehensive troubleshooting guide - Add production best practices and complete code examples Fixes google#1215
✅ Deploy Preview for adk-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Collaborator
|
Hey @perashanid. Thanks for the contribution. Due to the size of this article, there will be a substantial delay in reviewing the content before we decide we can accept it for publishing. In the meantime, we strongly encourage you to publish this content as a blog post or online developer forums for the benefit of developers using ADK. Thanks for sharing your knowledge with the ADK developer community! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds comprehensive documentation for implementing OpenTelemetry tracing with Vertex AI Agent Engine, addressing issue #1215.
The issue reported that documentation was insufficient to properly implement OpenTelemetry for Agent Engine, leading to blank dashboards and missing traces. The root cause was the lack of documentation about the critical initialization sequence - specifically that tracing MUST be initialized within the
AdkApp.set_up()method.Changes Made
New Documentation
docs/observability/tracing-agent-engine.md- Comprehensive guide covering:set_up()requirement)Updated Documentation
docs/observability/index.md- Updated to link to the new tracing guidemkdocs.yml- Added navigation entry for the new tracing pageTechnical Implementation
The documentation addresses all points raised in issue #1215:
✅ Documenting the init sequence clearly
set_up()is required✅ Documenting canonical data flows
✅ Architecture diagram of infrastructure configuration
✅ Reference to adk-samples
Key Features
Critical Warning Section
Added a prominent warning box at the top explaining the initialization requirement:
Two Implementation Paths
--trace_to_cloudflag orenable_tracing=TrueComprehensive Troubleshooting
Covers common issues:
Each issue includes symptoms and specific solutions.
Production Best Practices
Testing
Related Issue
Closes #1215
Checklist