fix(tracing): record span drop metrics in a single counter increment

Author: james-cardenas

src/agentex/lib/core/observability/tracing_metrics_recording.py

@@ -57,11 +57,13 @@ def record_span_enqueued(event_type: str) -> None:
         pass
 
 
-def record_span_dropped(reason: str) -> None:
-    if not is_metrics_enabled():
+def record_span_dropped(reason: str, count: int = 1) -> None:
+    if count <= 0 or not is_metrics_enabled():
         return
     try:
-        _tracing_module().get_tracing_metrics().span_events_dropped.add(1, {"reason": reason})
+        _tracing_module().get_tracing_metrics().span_events_dropped.add(
+            count, {"reason": reason}
+        )
     except Exception:
         pass
 

src/agentex/lib/core/tracing/span_queue.py

@@ -164,11 +164,9 @@ def _record_drop(self, count: int, reason: str) -> None:
             return
         self._dropped_spans += count
         if "shutting down" in reason:
-            for _ in range(count):
-                _metrics.record_span_dropped("shutdown")
+            _metrics.record_span_dropped("shutdown", count)
         elif "queue full" in reason:
-            for _ in range(count):
-                _metrics.record_span_dropped("queue_full")
+            _metrics.record_span_dropped("queue_full", count)
         # Warn on the first drop and then sparsely, so a drop storm is visible
         # without flooding the log.
         if self._dropped_spans == count or self._dropped_spans % 100 < count: