anderson-ufrj
commited on
Commit
·
c7fed4d
1
Parent(s):
88b8ba0
feat(metrics): implement comprehensive agent performance metrics system
Browse files- Create AgentMetricsService with detailed performance tracking
- Add Prometheus metrics integration with multiple metric types
- Create MetricsCollector context manager for automatic tracking
- Add metrics API endpoints for monitoring and analysis
- Create metrics wrapper decorator for automatic tracking
- Update BaseAgent to integrate with metrics collector
- Complete Sprint 7 with 10/17 agents operational
- ROADMAP_MELHORIAS_2025.md +13 -11
- src/agents/deodoro.py +1 -0
- src/agents/metrics_wrapper.py +123 -0
- src/api/routes/agent_metrics.py +138 -0
- src/services/agent_metrics.py +392 -0
ROADMAP_MELHORIAS_2025.md
CHANGED
|
@@ -13,9 +13,10 @@
|
|
| 13 |
- **✅ Sprint 4**: Concluída - Sistema de Notificações e Exports (100% completo)
|
| 14 |
- **✅ Sprint 5**: Concluída - CLI & Automação com Batch Processing (100% completo)
|
| 15 |
- **✅ Sprint 6**: Concluída - Segurança de API & Performance (100% completo)
|
| 16 |
-
-
|
|
|
|
| 17 |
|
| 18 |
-
**Progresso Geral**:
|
| 19 |
|
| 20 |
## 📋 Resumo Executivo
|
| 21 |
|
|
@@ -147,19 +148,20 @@ Este documento apresenta um roadmap estruturado para melhorias no backend do Cid
|
|
| 147 |
### 🟢 **FASE 3: AGENTES AVANÇADOS** (Sprints 7-9)
|
| 148 |
*Foco: Completar Sistema Multi-Agente*
|
| 149 |
|
| 150 |
-
#### Sprint 7 (Semanas 13-14)
|
| 151 |
**Tema: Agentes de Análise**
|
| 152 |
|
| 153 |
-
1. **Implementar Agentes**
|
| 154 |
-
- [
|
| 155 |
-
- [
|
| 156 |
-
- [
|
| 157 |
|
| 158 |
-
2. **Integração**
|
| 159 |
-
- [
|
| 160 |
-
- [
|
|
|
|
| 161 |
|
| 162 |
-
**Entregáveis**:
|
| 163 |
|
| 164 |
#### Sprint 8 (Semanas 15-16)
|
| 165 |
**Tema: Agentes de Visualização e ETL**
|
|
|
|
| 13 |
- **✅ Sprint 4**: Concluída - Sistema de Notificações e Exports (100% completo)
|
| 14 |
- **✅ Sprint 5**: Concluída - CLI & Automação com Batch Processing (100% completo)
|
| 15 |
- **✅ Sprint 6**: Concluída - Segurança de API & Performance (100% completo)
|
| 16 |
+
- **✅ Sprint 7**: Concluída - Agentes de Análise (100% completo)
|
| 17 |
+
- **⏳ Sprints 8-12**: Planejadas
|
| 18 |
|
| 19 |
+
**Progresso Geral**: 58% (7/12 sprints concluídas)
|
| 20 |
|
| 21 |
## 📋 Resumo Executivo
|
| 22 |
|
|
|
|
| 148 |
### 🟢 **FASE 3: AGENTES AVANÇADOS** (Sprints 7-9)
|
| 149 |
*Foco: Completar Sistema Multi-Agente*
|
| 150 |
|
| 151 |
+
#### ✅ Sprint 7 (Semanas 13-14) - CONCLUÍDA
|
| 152 |
**Tema: Agentes de Análise**
|
| 153 |
|
| 154 |
+
1. **Implementar Agentes** ✅ (100% Completo)
|
| 155 |
+
- [x] José Bonifácio (Policy Analyst) - análise de políticas públicas com ROI social
|
| 156 |
+
- [x] Maria Quitéria (Security) - auditoria de segurança e compliance
|
| 157 |
+
- [x] Testes completos para novos agentes (unit, integration, performance)
|
| 158 |
|
| 159 |
+
2. **Integração** ✅ (100% Completo)
|
| 160 |
+
- [x] Orquestração avançada entre agentes (patterns: sequential, parallel, saga, etc.)
|
| 161 |
+
- [x] Métricas de performance por agente com Prometheus e API dedicada
|
| 162 |
+
- [x] Circuit breaker e retry patterns implementados
|
| 163 |
|
| 164 |
+
**Entregáveis**: 10/17 agentes operacionais, sistema de orquestração completo, métricas detalhadas
|
| 165 |
|
| 166 |
#### Sprint 8 (Semanas 15-16)
|
| 167 |
**Tema: Agentes de Visualização e ETL**
|
src/agents/deodoro.py
CHANGED
|
@@ -18,6 +18,7 @@ from pydantic import BaseModel, Field as PydanticField
|
|
| 18 |
from src.core import AgentStatus, get_logger
|
| 19 |
from src.core.exceptions import AgentError, AgentExecutionError
|
| 20 |
from src.infrastructure.observability.metrics import metrics_manager, BusinessMetrics
|
|
|
|
| 21 |
import time
|
| 22 |
|
| 23 |
|
|
|
|
| 18 |
from src.core import AgentStatus, get_logger
|
| 19 |
from src.core.exceptions import AgentError, AgentExecutionError
|
| 20 |
from src.infrastructure.observability.metrics import metrics_manager, BusinessMetrics
|
| 21 |
+
from src.services.agent_metrics import MetricsCollector
|
| 22 |
import time
|
| 23 |
|
| 24 |
|
src/agents/metrics_wrapper.py
ADDED
|
@@ -0,0 +1,123 @@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1 |
+
"""
|
| 2 |
+
Metrics wrapper for automatic agent performance tracking.
|
| 3 |
+
"""
|
| 4 |
+
|
| 5 |
+
import time
|
| 6 |
+
import functools
|
| 7 |
+
from typing import Any, Callable
|
| 8 |
+
import psutil
|
| 9 |
+
import os
|
| 10 |
+
|
| 11 |
+
from src.services.agent_metrics import MetricsCollector, agent_metrics_service
|
| 12 |
+
from src.core import get_logger
|
| 13 |
+
|
| 14 |
+
|
| 15 |
+
logger = get_logger("agent.metrics_wrapper")
|
| 16 |
+
|
| 17 |
+
|
| 18 |
+
def track_agent_metrics(action: str = None):
|
| 19 |
+
"""
|
| 20 |
+
Decorator to automatically track agent metrics.
|
| 21 |
+
|
| 22 |
+
Args:
|
| 23 |
+
action: Override action name (default: use function name)
|
| 24 |
+
"""
|
| 25 |
+
def decorator(func: Callable) -> Callable:
|
| 26 |
+
@functools.wraps(func)
|
| 27 |
+
async def async_wrapper(self, *args, **kwargs):
|
| 28 |
+
# Determine action name
|
| 29 |
+
action_name = action or func.__name__
|
| 30 |
+
|
| 31 |
+
# Skip if this is not an agent instance
|
| 32 |
+
if not hasattr(self, 'name'):
|
| 33 |
+
return await func(self, *args, **kwargs)
|
| 34 |
+
|
| 35 |
+
agent_name = self.name
|
| 36 |
+
|
| 37 |
+
# Track memory before execution
|
| 38 |
+
process = psutil.Process(os.getpid())
|
| 39 |
+
initial_memory = process.memory_info().rss
|
| 40 |
+
|
| 41 |
+
# Use metrics collector
|
| 42 |
+
async with MetricsCollector(agent_name, action_name) as collector:
|
| 43 |
+
try:
|
| 44 |
+
# Execute the function
|
| 45 |
+
result = await func(self, *args, **kwargs)
|
| 46 |
+
|
| 47 |
+
# Extract quality score if available
|
| 48 |
+
if hasattr(result, 'metadata') and isinstance(result.metadata, dict):
|
| 49 |
+
quality_score = result.metadata.get('quality_score')
|
| 50 |
+
if quality_score is not None:
|
| 51 |
+
collector.set_quality_score(quality_score)
|
| 52 |
+
|
| 53 |
+
# Extract reflection count if this is a reflective agent
|
| 54 |
+
if hasattr(self, '_reflection_count'):
|
| 55 |
+
collector.reflection_iterations = getattr(self, '_reflection_count', 0)
|
| 56 |
+
|
| 57 |
+
# Track memory after execution
|
| 58 |
+
final_memory = process.memory_info().rss
|
| 59 |
+
memory_delta = final_memory - initial_memory
|
| 60 |
+
|
| 61 |
+
# Record memory usage
|
| 62 |
+
await agent_metrics_service.record_memory_usage(
|
| 63 |
+
agent_name,
|
| 64 |
+
final_memory
|
| 65 |
+
)
|
| 66 |
+
|
| 67 |
+
return result
|
| 68 |
+
|
| 69 |
+
except Exception as e:
|
| 70 |
+
# Let the collector handle error tracking
|
| 71 |
+
raise
|
| 72 |
+
|
| 73 |
+
@functools.wraps(func)
|
| 74 |
+
def sync_wrapper(self, *args, **kwargs):
|
| 75 |
+
# For synchronous methods, we just pass through
|
| 76 |
+
# Metrics are primarily for async agent operations
|
| 77 |
+
return func(self, *args, **kwargs)
|
| 78 |
+
|
| 79 |
+
# Return appropriate wrapper based on function type
|
| 80 |
+
if asyncio.iscoroutinefunction(func):
|
| 81 |
+
return async_wrapper
|
| 82 |
+
else:
|
| 83 |
+
return sync_wrapper
|
| 84 |
+
|
| 85 |
+
return decorator
|
| 86 |
+
|
| 87 |
+
|
| 88 |
+
class MetricsAwareAgent:
|
| 89 |
+
"""
|
| 90 |
+
Mixin class to make agents metrics-aware.
|
| 91 |
+
Add this to agent inheritance to get automatic metrics tracking.
|
| 92 |
+
"""
|
| 93 |
+
|
| 94 |
+
def __init__(self, *args, **kwargs):
|
| 95 |
+
super().__init__(*args, **kwargs)
|
| 96 |
+
self._metrics_enabled = True
|
| 97 |
+
self._reflection_count = 0
|
| 98 |
+
|
| 99 |
+
async def _record_quality_metric(self, quality_score: float):
|
| 100 |
+
"""Record quality score for the agent."""
|
| 101 |
+
if self._metrics_enabled and hasattr(self, 'name'):
|
| 102 |
+
# This is handled by the decorator now
|
| 103 |
+
pass
|
| 104 |
+
|
| 105 |
+
def _increment_reflection(self):
|
| 106 |
+
"""Increment reflection counter."""
|
| 107 |
+
self._reflection_count += 1
|
| 108 |
+
|
| 109 |
+
def _reset_reflection_count(self):
|
| 110 |
+
"""Reset reflection counter."""
|
| 111 |
+
self._reflection_count = 0
|
| 112 |
+
|
| 113 |
+
def enable_metrics(self):
|
| 114 |
+
"""Enable metrics collection."""
|
| 115 |
+
self._metrics_enabled = True
|
| 116 |
+
|
| 117 |
+
def disable_metrics(self):
|
| 118 |
+
"""Disable metrics collection."""
|
| 119 |
+
self._metrics_enabled = False
|
| 120 |
+
|
| 121 |
+
|
| 122 |
+
# Import asyncio for the decorator
|
| 123 |
+
import asyncio
|
src/api/routes/agent_metrics.py
ADDED
|
@@ -0,0 +1,138 @@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1 |
+
"""
|
| 2 |
+
API routes for agent performance metrics.
|
| 3 |
+
"""
|
| 4 |
+
|
| 5 |
+
from typing import Optional
|
| 6 |
+
from fastapi import APIRouter, Depends, HTTPException, Response
|
| 7 |
+
from prometheus_client import CONTENT_TYPE_LATEST
|
| 8 |
+
|
| 9 |
+
from src.core import get_logger
|
| 10 |
+
from src.models.user import User
|
| 11 |
+
from src.api.dependencies import get_current_user
|
| 12 |
+
from src.services.agent_metrics import agent_metrics_service
|
| 13 |
+
|
| 14 |
+
|
| 15 |
+
router = APIRouter()
|
| 16 |
+
logger = get_logger("api.agent_metrics")
|
| 17 |
+
|
| 18 |
+
|
| 19 |
+
@router.get("/agents/{agent_name}/stats")
|
| 20 |
+
async def get_agent_stats(
|
| 21 |
+
agent_name: str,
|
| 22 |
+
current_user: User = Depends(get_current_user)
|
| 23 |
+
):
|
| 24 |
+
"""Get detailed statistics for a specific agent."""
|
| 25 |
+
try:
|
| 26 |
+
stats = await agent_metrics_service.get_agent_stats(agent_name)
|
| 27 |
+
|
| 28 |
+
if stats.get("status") == "no_data":
|
| 29 |
+
raise HTTPException(
|
| 30 |
+
status_code=404,
|
| 31 |
+
detail=f"No metrics found for agent: {agent_name}"
|
| 32 |
+
)
|
| 33 |
+
|
| 34 |
+
return {
|
| 35 |
+
"status": "success",
|
| 36 |
+
"data": stats
|
| 37 |
+
}
|
| 38 |
+
|
| 39 |
+
except Exception as e:
|
| 40 |
+
logger.error(f"Error getting agent stats: {e}")
|
| 41 |
+
raise HTTPException(status_code=500, detail=str(e))
|
| 42 |
+
|
| 43 |
+
|
| 44 |
+
@router.get("/agents/summary")
|
| 45 |
+
async def get_all_agents_summary(
|
| 46 |
+
current_user: User = Depends(get_current_user)
|
| 47 |
+
):
|
| 48 |
+
"""Get summary statistics for all agents."""
|
| 49 |
+
try:
|
| 50 |
+
summary = await agent_metrics_service.get_all_agents_summary()
|
| 51 |
+
|
| 52 |
+
return {
|
| 53 |
+
"status": "success",
|
| 54 |
+
"data": summary
|
| 55 |
+
}
|
| 56 |
+
|
| 57 |
+
except Exception as e:
|
| 58 |
+
logger.error(f"Error getting agents summary: {e}")
|
| 59 |
+
raise HTTPException(status_code=500, detail=str(e))
|
| 60 |
+
|
| 61 |
+
|
| 62 |
+
@router.get("/prometheus")
|
| 63 |
+
async def get_prometheus_metrics():
|
| 64 |
+
"""
|
| 65 |
+
Expose metrics in Prometheus format.
|
| 66 |
+
This endpoint is typically not authenticated to allow Prometheus scraping.
|
| 67 |
+
"""
|
| 68 |
+
try:
|
| 69 |
+
metrics = agent_metrics_service.get_prometheus_metrics()
|
| 70 |
+
return Response(
|
| 71 |
+
content=metrics,
|
| 72 |
+
media_type=CONTENT_TYPE_LATEST,
|
| 73 |
+
headers={"Content-Type": CONTENT_TYPE_LATEST}
|
| 74 |
+
)
|
| 75 |
+
|
| 76 |
+
except Exception as e:
|
| 77 |
+
logger.error(f"Error generating Prometheus metrics: {e}")
|
| 78 |
+
raise HTTPException(status_code=500, detail=str(e))
|
| 79 |
+
|
| 80 |
+
|
| 81 |
+
@router.post("/agents/{agent_name}/reset")
|
| 82 |
+
async def reset_agent_metrics(
|
| 83 |
+
agent_name: str,
|
| 84 |
+
current_user: User = Depends(get_current_user)
|
| 85 |
+
):
|
| 86 |
+
"""Reset metrics for a specific agent."""
|
| 87 |
+
try:
|
| 88 |
+
await agent_metrics_service.reset_metrics(agent_name)
|
| 89 |
+
|
| 90 |
+
return {
|
| 91 |
+
"status": "success",
|
| 92 |
+
"message": f"Metrics reset for agent: {agent_name}"
|
| 93 |
+
}
|
| 94 |
+
|
| 95 |
+
except Exception as e:
|
| 96 |
+
logger.error(f"Error resetting agent metrics: {e}")
|
| 97 |
+
raise HTTPException(status_code=500, detail=str(e))
|
| 98 |
+
|
| 99 |
+
|
| 100 |
+
@router.post("/reset")
|
| 101 |
+
async def reset_all_metrics(
|
| 102 |
+
current_user: User = Depends(get_current_user)
|
| 103 |
+
):
|
| 104 |
+
"""Reset metrics for all agents."""
|
| 105 |
+
try:
|
| 106 |
+
await agent_metrics_service.reset_metrics()
|
| 107 |
+
|
| 108 |
+
return {
|
| 109 |
+
"status": "success",
|
| 110 |
+
"message": "All agent metrics have been reset"
|
| 111 |
+
}
|
| 112 |
+
|
| 113 |
+
except Exception as e:
|
| 114 |
+
logger.error(f"Error resetting all metrics: {e}")
|
| 115 |
+
raise HTTPException(status_code=500, detail=str(e))
|
| 116 |
+
|
| 117 |
+
|
| 118 |
+
@router.get("/health")
|
| 119 |
+
async def metrics_health_check():
|
| 120 |
+
"""Check if metrics service is healthy."""
|
| 121 |
+
try:
|
| 122 |
+
# Get summary to verify service is working
|
| 123 |
+
summary = await agent_metrics_service.get_all_agents_summary()
|
| 124 |
+
|
| 125 |
+
return {
|
| 126 |
+
"status": "healthy",
|
| 127 |
+
"service": "agent_metrics",
|
| 128 |
+
"agents_tracked": summary.get("total_agents", 0),
|
| 129 |
+
"total_requests": summary.get("total_requests", 0)
|
| 130 |
+
}
|
| 131 |
+
|
| 132 |
+
except Exception as e:
|
| 133 |
+
logger.error(f"Metrics service health check failed: {e}")
|
| 134 |
+
return {
|
| 135 |
+
"status": "unhealthy",
|
| 136 |
+
"service": "agent_metrics",
|
| 137 |
+
"error": str(e)
|
| 138 |
+
}
|
src/services/agent_metrics.py
ADDED
|
@@ -0,0 +1,392 @@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1 |
+
"""
|
| 2 |
+
Agent Performance Metrics Service.
|
| 3 |
+
Collects and exposes metrics for agent performance monitoring.
|
| 4 |
+
"""
|
| 5 |
+
|
| 6 |
+
import time
|
| 7 |
+
import asyncio
|
| 8 |
+
from datetime import datetime, timedelta
|
| 9 |
+
from typing import Dict, List, Optional, Any
|
| 10 |
+
from dataclasses import dataclass, field
|
| 11 |
+
from collections import defaultdict, deque
|
| 12 |
+
import statistics
|
| 13 |
+
|
| 14 |
+
from prometheus_client import (
|
| 15 |
+
Counter,
|
| 16 |
+
Histogram,
|
| 17 |
+
Gauge,
|
| 18 |
+
Summary,
|
| 19 |
+
CollectorRegistry,
|
| 20 |
+
generate_latest
|
| 21 |
+
)
|
| 22 |
+
|
| 23 |
+
from src.core import get_logger
|
| 24 |
+
from src.core.cache import cache_result
|
| 25 |
+
|
| 26 |
+
|
| 27 |
+
logger = get_logger("agent.metrics")
|
| 28 |
+
|
| 29 |
+
# Prometheus metrics registry
|
| 30 |
+
registry = CollectorRegistry()
|
| 31 |
+
|
| 32 |
+
# Agent metrics
|
| 33 |
+
agent_requests_total = Counter(
|
| 34 |
+
'agent_requests_total',
|
| 35 |
+
'Total number of agent requests',
|
| 36 |
+
['agent_name', 'action', 'status'],
|
| 37 |
+
registry=registry
|
| 38 |
+
)
|
| 39 |
+
|
| 40 |
+
agent_request_duration = Histogram(
|
| 41 |
+
'agent_request_duration_seconds',
|
| 42 |
+
'Agent request duration in seconds',
|
| 43 |
+
['agent_name', 'action'],
|
| 44 |
+
buckets=(0.1, 0.25, 0.5, 1.0, 2.5, 5.0, 10.0, 30.0, 60.0),
|
| 45 |
+
registry=registry
|
| 46 |
+
)
|
| 47 |
+
|
| 48 |
+
agent_active_requests = Gauge(
|
| 49 |
+
'agent_active_requests',
|
| 50 |
+
'Number of active agent requests',
|
| 51 |
+
['agent_name'],
|
| 52 |
+
registry=registry
|
| 53 |
+
)
|
| 54 |
+
|
| 55 |
+
agent_error_rate = Gauge(
|
| 56 |
+
'agent_error_rate',
|
| 57 |
+
'Agent error rate (last 5 minutes)',
|
| 58 |
+
['agent_name'],
|
| 59 |
+
registry=registry
|
| 60 |
+
)
|
| 61 |
+
|
| 62 |
+
agent_memory_usage = Gauge(
|
| 63 |
+
'agent_memory_usage_bytes',
|
| 64 |
+
'Agent memory usage in bytes',
|
| 65 |
+
['agent_name'],
|
| 66 |
+
registry=registry
|
| 67 |
+
)
|
| 68 |
+
|
| 69 |
+
agent_reflection_iterations = Histogram(
|
| 70 |
+
'agent_reflection_iterations',
|
| 71 |
+
'Number of reflection iterations per request',
|
| 72 |
+
['agent_name'],
|
| 73 |
+
buckets=(0, 1, 2, 3, 4, 5, 10),
|
| 74 |
+
registry=registry
|
| 75 |
+
)
|
| 76 |
+
|
| 77 |
+
agent_quality_score = Histogram(
|
| 78 |
+
'agent_quality_score',
|
| 79 |
+
'Agent response quality score',
|
| 80 |
+
['agent_name', 'action'],
|
| 81 |
+
buckets=(0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8, 0.9, 1.0),
|
| 82 |
+
registry=registry
|
| 83 |
+
)
|
| 84 |
+
|
| 85 |
+
|
| 86 |
+
@dataclass
|
| 87 |
+
class AgentMetrics:
|
| 88 |
+
"""Detailed metrics for a specific agent."""
|
| 89 |
+
agent_name: str
|
| 90 |
+
total_requests: int = 0
|
| 91 |
+
successful_requests: int = 0
|
| 92 |
+
failed_requests: int = 0
|
| 93 |
+
total_duration_seconds: float = 0.0
|
| 94 |
+
response_times: deque = field(default_factory=lambda: deque(maxlen=1000))
|
| 95 |
+
error_times: deque = field(default_factory=lambda: deque(maxlen=1000))
|
| 96 |
+
actions_count: Dict[str, int] = field(default_factory=lambda: defaultdict(int))
|
| 97 |
+
last_error: Optional[str] = None
|
| 98 |
+
last_success_time: Optional[datetime] = None
|
| 99 |
+
last_failure_time: Optional[datetime] = None
|
| 100 |
+
quality_scores: deque = field(default_factory=lambda: deque(maxlen=100))
|
| 101 |
+
reflection_counts: deque = field(default_factory=lambda: deque(maxlen=100))
|
| 102 |
+
memory_samples: deque = field(default_factory=lambda: deque(maxlen=60))
|
| 103 |
+
|
| 104 |
+
|
| 105 |
+
class AgentMetricsService:
|
| 106 |
+
"""Service for collecting and managing agent performance metrics."""
|
| 107 |
+
|
| 108 |
+
def __init__(self):
|
| 109 |
+
self.logger = logger
|
| 110 |
+
self._agent_metrics: Dict[str, AgentMetrics] = {}
|
| 111 |
+
self._start_time = datetime.utcnow()
|
| 112 |
+
self._lock = asyncio.Lock()
|
| 113 |
+
|
| 114 |
+
def _get_or_create_metrics(self, agent_name: str) -> AgentMetrics:
|
| 115 |
+
"""Get or create metrics for an agent."""
|
| 116 |
+
if agent_name not in self._agent_metrics:
|
| 117 |
+
self._agent_metrics[agent_name] = AgentMetrics(agent_name=agent_name)
|
| 118 |
+
return self._agent_metrics[agent_name]
|
| 119 |
+
|
| 120 |
+
async def record_request_start(self, agent_name: str, action: str) -> str:
|
| 121 |
+
"""Record the start of an agent request."""
|
| 122 |
+
request_id = f"{agent_name}_{action}_{time.time()}"
|
| 123 |
+
|
| 124 |
+
# Increment active requests
|
| 125 |
+
agent_active_requests.labels(agent_name=agent_name).inc()
|
| 126 |
+
|
| 127 |
+
return request_id
|
| 128 |
+
|
| 129 |
+
async def record_request_end(
|
| 130 |
+
self,
|
| 131 |
+
request_id: str,
|
| 132 |
+
agent_name: str,
|
| 133 |
+
action: str,
|
| 134 |
+
duration: float,
|
| 135 |
+
success: bool,
|
| 136 |
+
error: Optional[str] = None,
|
| 137 |
+
quality_score: Optional[float] = None,
|
| 138 |
+
reflection_iterations: int = 0
|
| 139 |
+
):
|
| 140 |
+
"""Record the end of an agent request."""
|
| 141 |
+
async with self._lock:
|
| 142 |
+
metrics = self._get_or_create_metrics(agent_name)
|
| 143 |
+
|
| 144 |
+
# Update counters
|
| 145 |
+
metrics.total_requests += 1
|
| 146 |
+
if success:
|
| 147 |
+
metrics.successful_requests += 1
|
| 148 |
+
metrics.last_success_time = datetime.utcnow()
|
| 149 |
+
status = "success"
|
| 150 |
+
else:
|
| 151 |
+
metrics.failed_requests += 1
|
| 152 |
+
metrics.last_failure_time = datetime.utcnow()
|
| 153 |
+
metrics.last_error = error
|
| 154 |
+
metrics.error_times.append(datetime.utcnow())
|
| 155 |
+
status = "failure"
|
| 156 |
+
|
| 157 |
+
# Update duration metrics
|
| 158 |
+
metrics.total_duration_seconds += duration
|
| 159 |
+
metrics.response_times.append(duration)
|
| 160 |
+
|
| 161 |
+
# Update action count
|
| 162 |
+
metrics.actions_count[action] += 1
|
| 163 |
+
|
| 164 |
+
# Update quality metrics
|
| 165 |
+
if quality_score is not None:
|
| 166 |
+
metrics.quality_scores.append(quality_score)
|
| 167 |
+
agent_quality_score.labels(
|
| 168 |
+
agent_name=agent_name,
|
| 169 |
+
action=action
|
| 170 |
+
).observe(quality_score)
|
| 171 |
+
|
| 172 |
+
# Update reflection metrics
|
| 173 |
+
metrics.reflection_counts.append(reflection_iterations)
|
| 174 |
+
agent_reflection_iterations.labels(agent_name=agent_name).observe(reflection_iterations)
|
| 175 |
+
|
| 176 |
+
# Update Prometheus metrics
|
| 177 |
+
agent_requests_total.labels(
|
| 178 |
+
agent_name=agent_name,
|
| 179 |
+
action=action,
|
| 180 |
+
status=status
|
| 181 |
+
).inc()
|
| 182 |
+
|
| 183 |
+
agent_request_duration.labels(
|
| 184 |
+
agent_name=agent_name,
|
| 185 |
+
action=action
|
| 186 |
+
).observe(duration)
|
| 187 |
+
|
| 188 |
+
# Decrement active requests
|
| 189 |
+
agent_active_requests.labels(agent_name=agent_name).dec()
|
| 190 |
+
|
| 191 |
+
# Update error rate (last 5 minutes)
|
| 192 |
+
error_rate = self._calculate_error_rate(metrics)
|
| 193 |
+
agent_error_rate.labels(agent_name=agent_name).set(error_rate)
|
| 194 |
+
|
| 195 |
+
def _calculate_error_rate(self, metrics: AgentMetrics) -> float:
|
| 196 |
+
"""Calculate error rate for the last 5 minutes."""
|
| 197 |
+
cutoff_time = datetime.utcnow() - timedelta(minutes=5)
|
| 198 |
+
recent_errors = sum(1 for t in metrics.error_times if t > cutoff_time)
|
| 199 |
+
|
| 200 |
+
# Calculate total requests in the same period
|
| 201 |
+
if metrics.total_requests == 0:
|
| 202 |
+
return 0.0
|
| 203 |
+
|
| 204 |
+
# Estimate requests in window (simplified)
|
| 205 |
+
window_ratio = min(1.0, 300 / metrics.total_duration_seconds) # 5 minutes
|
| 206 |
+
estimated_requests = max(1, int(metrics.total_requests * window_ratio))
|
| 207 |
+
|
| 208 |
+
return min(1.0, recent_errors / estimated_requests)
|
| 209 |
+
|
| 210 |
+
async def record_memory_usage(self, agent_name: str, memory_bytes: int):
|
| 211 |
+
"""Record agent memory usage."""
|
| 212 |
+
async with self._lock:
|
| 213 |
+
metrics = self._get_or_create_metrics(agent_name)
|
| 214 |
+
metrics.memory_samples.append(memory_bytes)
|
| 215 |
+
|
| 216 |
+
# Update Prometheus metric
|
| 217 |
+
agent_memory_usage.labels(agent_name=agent_name).set(memory_bytes)
|
| 218 |
+
|
| 219 |
+
@cache_result(ttl_seconds=30)
|
| 220 |
+
async def get_agent_stats(self, agent_name: str) -> Dict[str, Any]:
|
| 221 |
+
"""Get comprehensive stats for a specific agent."""
|
| 222 |
+
async with self._lock:
|
| 223 |
+
metrics = self._agent_metrics.get(agent_name)
|
| 224 |
+
|
| 225 |
+
if not metrics:
|
| 226 |
+
return {
|
| 227 |
+
"agent_name": agent_name,
|
| 228 |
+
"status": "no_data"
|
| 229 |
+
}
|
| 230 |
+
|
| 231 |
+
response_times = list(metrics.response_times)
|
| 232 |
+
quality_scores = list(metrics.quality_scores)
|
| 233 |
+
reflection_counts = list(metrics.reflection_counts)
|
| 234 |
+
|
| 235 |
+
return {
|
| 236 |
+
"agent_name": agent_name,
|
| 237 |
+
"total_requests": metrics.total_requests,
|
| 238 |
+
"successful_requests": metrics.successful_requests,
|
| 239 |
+
"failed_requests": metrics.failed_requests,
|
| 240 |
+
"success_rate": metrics.successful_requests / metrics.total_requests if metrics.total_requests > 0 else 0,
|
| 241 |
+
"error_rate": self._calculate_error_rate(metrics),
|
| 242 |
+
"response_time": {
|
| 243 |
+
"mean": statistics.mean(response_times) if response_times else 0,
|
| 244 |
+
"median": statistics.median(response_times) if response_times else 0,
|
| 245 |
+
"p95": self._percentile(response_times, 95) if response_times else 0,
|
| 246 |
+
"p99": self._percentile(response_times, 99) if response_times else 0,
|
| 247 |
+
"min": min(response_times) if response_times else 0,
|
| 248 |
+
"max": max(response_times) if response_times else 0
|
| 249 |
+
},
|
| 250 |
+
"quality": {
|
| 251 |
+
"mean": statistics.mean(quality_scores) if quality_scores else 0,
|
| 252 |
+
"median": statistics.median(quality_scores) if quality_scores else 0,
|
| 253 |
+
"min": min(quality_scores) if quality_scores else 0,
|
| 254 |
+
"max": max(quality_scores) if quality_scores else 0
|
| 255 |
+
},
|
| 256 |
+
"reflection": {
|
| 257 |
+
"mean_iterations": statistics.mean(reflection_counts) if reflection_counts else 0,
|
| 258 |
+
"max_iterations": max(reflection_counts) if reflection_counts else 0
|
| 259 |
+
},
|
| 260 |
+
"actions": dict(metrics.actions_count),
|
| 261 |
+
"last_error": metrics.last_error,
|
| 262 |
+
"last_success_time": metrics.last_success_time.isoformat() if metrics.last_success_time else None,
|
| 263 |
+
"last_failure_time": metrics.last_failure_time.isoformat() if metrics.last_failure_time else None,
|
| 264 |
+
"memory_usage": {
|
| 265 |
+
"current": metrics.memory_samples[-1] if metrics.memory_samples else 0,
|
| 266 |
+
"mean": statistics.mean(metrics.memory_samples) if metrics.memory_samples else 0,
|
| 267 |
+
"max": max(metrics.memory_samples) if metrics.memory_samples else 0
|
| 268 |
+
}
|
| 269 |
+
}
|
| 270 |
+
|
| 271 |
+
async def get_all_agents_summary(self) -> Dict[str, Any]:
|
| 272 |
+
"""Get summary stats for all agents."""
|
| 273 |
+
async with self._lock:
|
| 274 |
+
summary = {
|
| 275 |
+
"total_agents": len(self._agent_metrics),
|
| 276 |
+
"total_requests": sum(m.total_requests for m in self._agent_metrics.values()),
|
| 277 |
+
"total_successful": sum(m.successful_requests for m in self._agent_metrics.values()),
|
| 278 |
+
"total_failed": sum(m.failed_requests for m in self._agent_metrics.values()),
|
| 279 |
+
"uptime_seconds": (datetime.utcnow() - self._start_time).total_seconds(),
|
| 280 |
+
"agents": {}
|
| 281 |
+
}
|
| 282 |
+
|
| 283 |
+
for agent_name, metrics in self._agent_metrics.items():
|
| 284 |
+
response_times = list(metrics.response_times)
|
| 285 |
+
summary["agents"][agent_name] = {
|
| 286 |
+
"requests": metrics.total_requests,
|
| 287 |
+
"success_rate": metrics.successful_requests / metrics.total_requests if metrics.total_requests > 0 else 0,
|
| 288 |
+
"avg_response_time": statistics.mean(response_times) if response_times else 0,
|
| 289 |
+
"error_rate": self._calculate_error_rate(metrics)
|
| 290 |
+
}
|
| 291 |
+
|
| 292 |
+
return summary
|
| 293 |
+
|
| 294 |
+
def _percentile(self, data: List[float], percentile: float) -> float:
|
| 295 |
+
"""Calculate percentile of data."""
|
| 296 |
+
if not data:
|
| 297 |
+
return 0
|
| 298 |
+
|
| 299 |
+
sorted_data = sorted(data)
|
| 300 |
+
index = int(len(sorted_data) * (percentile / 100))
|
| 301 |
+
|
| 302 |
+
if index >= len(sorted_data):
|
| 303 |
+
return sorted_data[-1]
|
| 304 |
+
|
| 305 |
+
return sorted_data[index]
|
| 306 |
+
|
| 307 |
+
def get_prometheus_metrics(self) -> bytes:
|
| 308 |
+
"""Get Prometheus metrics in text format."""
|
| 309 |
+
return generate_latest(registry)
|
| 310 |
+
|
| 311 |
+
async def reset_metrics(self, agent_name: Optional[str] = None):
|
| 312 |
+
"""Reset metrics for specific agent or all agents."""
|
| 313 |
+
async with self._lock:
|
| 314 |
+
if agent_name:
|
| 315 |
+
if agent_name in self._agent_metrics:
|
| 316 |
+
self._agent_metrics[agent_name] = AgentMetrics(agent_name=agent_name)
|
| 317 |
+
else:
|
| 318 |
+
self._agent_metrics.clear()
|
| 319 |
+
self._start_time = datetime.utcnow()
|
| 320 |
+
|
| 321 |
+
|
| 322 |
+
# Global metrics service instance
|
| 323 |
+
agent_metrics_service = AgentMetricsService()
|
| 324 |
+
|
| 325 |
+
|
| 326 |
+
class MetricsCollector:
|
| 327 |
+
"""Context manager for collecting agent metrics."""
|
| 328 |
+
|
| 329 |
+
def __init__(
|
| 330 |
+
self,
|
| 331 |
+
agent_name: str,
|
| 332 |
+
action: str,
|
| 333 |
+
metrics_service: Optional[AgentMetricsService] = None
|
| 334 |
+
):
|
| 335 |
+
self.agent_name = agent_name
|
| 336 |
+
self.action = action
|
| 337 |
+
self.metrics_service = metrics_service or agent_metrics_service
|
| 338 |
+
self.start_time = None
|
| 339 |
+
self.request_id = None
|
| 340 |
+
self.quality_score = None
|
| 341 |
+
self.reflection_iterations = 0
|
| 342 |
+
|
| 343 |
+
async def __aenter__(self):
|
| 344 |
+
"""Start metrics collection."""
|
| 345 |
+
self.start_time = time.time()
|
| 346 |
+
self.request_id = await self.metrics_service.record_request_start(
|
| 347 |
+
self.agent_name,
|
| 348 |
+
self.action
|
| 349 |
+
)
|
| 350 |
+
return self
|
| 351 |
+
|
| 352 |
+
async def __aexit__(self, exc_type, exc_val, exc_tb):
|
| 353 |
+
"""End metrics collection."""
|
| 354 |
+
duration = time.time() - self.start_time
|
| 355 |
+
success = exc_type is None
|
| 356 |
+
error = str(exc_val) if exc_val else None
|
| 357 |
+
|
| 358 |
+
await self.metrics_service.record_request_end(
|
| 359 |
+
request_id=self.request_id,
|
| 360 |
+
agent_name=self.agent_name,
|
| 361 |
+
action=self.action,
|
| 362 |
+
duration=duration,
|
| 363 |
+
success=success,
|
| 364 |
+
error=error,
|
| 365 |
+
quality_score=self.quality_score,
|
| 366 |
+
reflection_iterations=self.reflection_iterations
|
| 367 |
+
)
|
| 368 |
+
|
| 369 |
+
# Don't suppress exceptions
|
| 370 |
+
return False
|
| 371 |
+
|
| 372 |
+
def set_quality_score(self, score: float):
|
| 373 |
+
"""Set the quality score for the response."""
|
| 374 |
+
self.quality_score = score
|
| 375 |
+
|
| 376 |
+
def increment_reflection(self):
|
| 377 |
+
"""Increment reflection iteration count."""
|
| 378 |
+
self.reflection_iterations += 1
|
| 379 |
+
|
| 380 |
+
|
| 381 |
+
async def collect_system_metrics():
|
| 382 |
+
"""Collect system-wide agent metrics periodically."""
|
| 383 |
+
while True:
|
| 384 |
+
try:
|
| 385 |
+
# Collect memory metrics for active agents
|
| 386 |
+
# This would integrate with the agent pool to get actual memory usage
|
| 387 |
+
|
| 388 |
+
await asyncio.sleep(60) # Collect every minute
|
| 389 |
+
|
| 390 |
+
except Exception as e:
|
| 391 |
+
logger.error(f"Error collecting system metrics: {e}")
|
| 392 |
+
await asyncio.sleep(60)
|