go-libp2p icon indicating copy to clipboard operation
go-libp2p copied to clipboard

rcmgr: centralize blocked resource logging through trace reporter

Open sneaxhuh opened this issue 4 weeks ago • 0 comments

Fixes #1744

This change consolidates debug logging for blocked resources into a centralized trace reporter system. Previously, debug logs for blocked resources were scattered throughout the codebase and didn't always align with the metrics, which were reliably tracked through trace events.

Changes:

  • Add LogReporter that consumes trace events and logs blocked resources
  • Remove ad-hoc log.Debug() calls from scope.go
  • Install LogReporter by default in NewResourceManager
  • Add tests for LogReporter functionality

Benefits:

  • Logs and metrics now use the same source (trace events)
  • More maintainable centralized logging approach
  • Consistent blocked resource reporting

sneaxhuh avatar Nov 30 '25 08:11 sneaxhuh