Skip to content

feat: reduce verbose debug logging across all instrumentation modules #1095

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Reduce Verbose Debug Logging Across All Instrumentation Modules

Overview

This PR comprehensively reduces verbose debug logging across all AgentOps SDK instrumentation modules, addressing the issue identified by Dwij1704 that debug logs exist beyond just the Google ADK integration and need cleanup across all instrumentations.

Changes Made

Debug Log Cleanup Strategy

  • Removed operational debug logs that provide no value to users (method wrapping success, package detection, status messages)
  • Converted error condition debug logs to warning level for better visibility during troubleshooting
  • Preserved essential error reporting and functionality while reducing log noise
  • Maintained all existing functionality with improved user experience

Files Modified (28 total)

  • Main instrumentation module (__init__.py) - Removed package detection and instrumentation status debug logs
  • Agentic instrumentations: AG2, OpenAI Agents, Google ADK, Agno, SmoLAgents
  • Provider instrumentations: OpenAI, Anthropic, Google GenAI, IBM WatsonX AI, Mem0
  • Common utilities: instrumentor, token counting, version detection, span management
  • Stream wrappers and attribute processors across all providers

Impact

  • Significantly improved user experience by reducing console log noise
  • Better error visibility by converting critical debug logs to warning level
  • Maintained all functionality while providing cleaner output
  • Consistent logging approach across all instrumentation modules

Before/After Example

Before:

DEBUG: Successfully wrapped ConversableAgent.__init__
DEBUG: Module 'openai' is a library, instrumenting 'openai'
DEBUG: Applying Google ADK patches for AgentOps instrumentation
DEBUG: Replaced tracer in google.adk.core

After:

INFO: OpenAI instrumentation enabled
INFO: Google ADK instrumentation enabled

Testing

  • ✅ All pre-commit checks pass
  • ✅ Code formatting and linting applied automatically
  • ✅ No functional changes to instrumentation behavior
  • ✅ Comprehensive search confirms debug log removal across all modules

Related Issues


Link to Devin run: https://app.devin.ai/sessions/cf63459ec9c1488e82bb0d2de491a11e
Requested by: Alex ([email protected])

- Remove operational debug logs that provide no value to users
- Convert error condition debug logs to warning level for visibility
- Remove package detection, method wrapping, and status debug logs
- Preserve essential error reporting and functionality
- Affects all instrumentation modules: agentic, providers, utilities
- Significantly improves user experience by reducing log noise

Addresses comprehensive debug logging cleanup as requested by Dwij1704

Co-Authored-By: Alex <[email protected]>
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.

0 participants