The two new counters are rendered under each other's names
Defect 13 / Tier 3 / issue
services/proxy/src/metrics.rs:50-57services/proxy/src/metrics.rs:94-95Expected review
put the two arrays in the same order - the existing comment above `COUNTERS` already says that is the invariant.
Discovery difficulty
each array is internally consistent and each ordering is defensible alone; `values()` matches the struct and `COUNTERS` matches the order somebody would describe the counters in. The coupling lives in `render`, which the diff does not touch. The comment naming this exact failure is four lines above one of the two arrays, which makes it findable and also makes it read as already handled.
Trigger
any sweep. Verified: `record_sweep(3, 4096)` renders `cairn_proxy_sweep_blobs_removed_total 4096` and `cairn_proxy_sweep_bytes_reclaimed_total 3`. Every dashboard and alert on either number reads the other, and `docs/operations.md` tells operators to watch `cairn_proxy_sweep_bytes_reclaimed_total` by name.
Severity
issue