Cooper Ry Lees 4308eb1fa5 Also prune response-time histogram "check" call_type on peer removal ...

Address PR #169 review: DeletePeerMetrics only removed the "ping"
call_type label set, leaving stale "check" samples from CheckAllPods
in /metrics after a peer rolls. goldpingerResponseTimePeersHistogram
is observed at two sites — Pinger (pinger.go:56, "ping") and
CheckAllPods via GetLabeledPeersCallsTimer (client.go:209, "check")
— so both must be pruned.

Missed on the first pass because I traced the Pinger observation
(the hot path) and did not separately check for the "check" label
value emitted from the /check_all handler. Tests now seed both
call_types before teardown so either gap regresses.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Cooper Ry Lees <me@cooperlees.com>

2026-04-22 13:54:01 +00:00

..

client

Add UDP probe metrics: packet loss, hop count, and RTT

2026-03-27 16:05:32 +00:00

goldpinger

Also prune response-time histogram "check" call_type on peer removal

2026-04-22 13:54:01 +00:00

models

Rename PathLength to HopCount in swagger model and UI

2026-04-02 19:45:31 +00:00

restapi

Rename PathLength to HopCount in swagger model and UI

2026-04-02 19:45:31 +00:00