Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Reduce verbose debug logging in Google ADK integration

Summary

This PR addresses the issue of excessive debug logging in the Google ADK integration that was creating verbose output beyond just the trace URL printing.

Changes Made

  • Removed verbose debug logs for individual method wrapping operations that were cluttering the output
  • Removed debug logs for tracer replacement operations that provided unnecessary detail
  • Changed essential status messages from debug to info level for better visibility:
    • "Applying Google ADK patches for AgentOps instrumentation"
    • "Removing Google ADK patches"
  • Preserved all warning and error logs for important diagnostic information
  • Kept essential status logs like "Google ADK instrumentation enabled/disabled" and "Google ADK patching complete"

What's Fixed

Before this change, users would see verbose debug output like:

  • "Successfully wrapped google.adk.agents.base_agent.BaseAgent.run_async"
  • "Replaced ADK's tracer with NoOpTracer"
  • "Replaced tracer in google.adk.runners"
  • Multiple similar wrapping success messages

After this change, users will only see essential status information while maintaining full functionality.

Testing

  • ✅ Trace URL functionality remains intact (handled by core AgentOps system)
  • ✅ Google ADK instrumentation continues to work correctly
  • ✅ Essential status messages are still visible for debugging when needed
  • ✅ Verbose operational details are no longer cluttering the output

Files Modified

  • agentops/instrumentation/google_adk/patch.py - Removed 8 verbose debug log statements

Impact

  • Reduced log noise during Google ADK instrumentation setup
  • Improved user experience with cleaner console output
  • Maintained functionality - all instrumentation features work as before
  • Preserved essential logging for troubleshooting when needed

Link to Devin run: https://app.devin.ai/sessions/cf63459ec9c1488e82bb0d2de491a11e

Requested by: Alex ([email protected])

- Remove verbose debug logs for individual wrapping operations
- Remove debug logs for tracer replacement operations
- Change essential patch status messages from debug to info level
- Keep warning/error logs and essential status messages intact
- Maintains trace URL functionality while reducing log noise

Fixes verbose output during Google ADK instrumentation setup

Co-Authored-By: Alex <[email protected]>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@codecov
Copy link

codecov bot commented Jun 18, 2025

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
agentops/instrumentation/google_adk/patch.py 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@areibman areibman requested a review from Dwij1704 June 19, 2025 00:58
@areibman
Copy link
Contributor

We forgot some logs @Dwij1704. Easy merge! :)

- Change 'Failed to extract response attributes' from debug to warning
- This ensures error conditions are still visible while reducing debug noise
- Addresses feedback from PR review about missed debug logs

Co-Authored-By: Alex <[email protected]>
@Dwij1704
Copy link
Member

We forgot some logs @Dwij1704. Easy merge! :)

Will create a separate PR for this, I see a lot of logs in other instrumentations too

@devin-ai-integration
Copy link
Contributor Author

Closing this PR to create a comprehensive solution that addresses debug logs across all instrumentation modules, not just Google ADK.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants