W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > March 2020

Re: [webrtc-stats] Add counters for bytes sent by ICE pings (#534) (#535)

From: Jonas Oreland via GitHub <sysbot+gh@w3.org>
Date: Wed, 25 Mar 2020 13:54:39 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-603852552-1585144477-sysbot+gh@w3.org>
1) I don't think it adds value to talk about ICE protocol header...just bytes,
i.e the metric should include all bytes sent as STUN pings.

2) "ICE ping packed used for probing", parse error.
The RTP probing is done using RTP packets.

3) "Or are we sending lots and lots of tiny ICE pings but they add up to a lot in the end?"
Sort of yes, the ICE agent that I'm working on constraints ICE pings to X bits/sec.
Bitrate for RTP is sufficiently small, then X is noticable.

4) I added this cause I've changed the frequency of the ICE pings,
and there was no real way to measure that.

GitHub Notification of comment by jonasore
Please view or discuss this issue at https://github.com/w3c/webrtc-stats/pull/535#issuecomment-603852552 using your GitHub account
Received on Wednesday, 25 March 2020 13:54:41 UTC

This archive was generated by hypermail 2.4.0 : Saturday, 6 May 2023 21:19:50 UTC