mirror of
https://github.com/netbirdio/netbird.git
synced 2026-08-25 00:51:28 +02:00
b339917b93
recordConnectionMetrics mapped only conntype.Relay to "relay" and let a default branch record everything else as "ice". ICETurn (ICE through a TURN server, which conn.isRelayed treats as relayed) and None (no connection established yet, set on relay drop and on peer-state reset) were therefore reported as "ice" but a fraction of it was not p2p was over a turn server so in a sense "relayed". turn has more affinity with "relayed" than with "p2p", because if you didn't have turn server, you'd end up relayed.. so it's giving you information on how much you'd depend on relayed conns The mapping now lists every priority explicitly and emits ice_p2p, ice_turn or relay. It skips "None" (unknown) conn type. Values do not reuse "ice": samples recorded with the old tag conflate the three states, so keeping the name would make historical and corrected samples indistinguishable.