Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add RTT to Sum when using JSON #1724

Open
hajoha opened this issue Jul 3, 2024 · 0 comments
Open

Add RTT to Sum when using JSON #1724

hajoha opened this issue Jul 3, 2024 · 0 comments

Comments

@hajoha
Copy link

hajoha commented Jul 3, 2024

Context

  • Version of iperf3:
    all

  • Operating system (and distribution, if any):
    all

Enhancement Request

  • Current behavior
    Currently the RTT stats are not shown in the Sum Object
	"streams":	[{
			"socket":	6,
			"start":	1.001069,
			"end":	2.000895,
			"seconds":	0.99982601404190063,
			"bytes":	11927552,
			"bits_per_second":	95437020.7014849,
			"retransmits":	15,
			"snd_cwnd":	139152,
			"snd_wnd":	3142272,
			"rtt":	13505,
			"rttvar":	994,
			"pmtu":	1422,
			"omitted":	false,
			"sender":	true
		}],
	"sum":	{
		"start":	1.001069,
		"end":	2.000895,
		"seconds":	0.99982601404190063,
		"bytes":	11927552,
		"bits_per_second":	95437020.7014849,
		"retransmits":	15,
		"omitted":	false,
		"sender":	true
	}
  • Desired behavior
    Display the RTT stats for each interval also in the sum object
	"streams":	[{
			"socket":	6,
			"start":	1.001069,
			"end":	2.000895,
			"seconds":	0.99982601404190063,
			"bytes":	11927552,
			"bits_per_second":	95437020.7014849,
			"retransmits":	15,
			"snd_cwnd":	139152,
			"snd_wnd":	3142272,
			"rtt":	13505,
			"rttvar":	994,
			"pmtu":	1422,
			"omitted":	false,
			"sender":	true
		}],
	"sum":	{
		"start":	1.001069,
		"end":	2.000895,
		"seconds":	0.99982601404190063,
		"bytes":	11927552,
		"bits_per_second":	95437020.7014849,
		"retransmits":	15,
		"omitted":	false,
		"rtt":	13505,
		"rttvar":	994,
		"sender":	true
	}
  • Implementation notes
    I'm curious why this hasn't been implemented yet. Is it because the mean RTT for the streams is considered useless, or has it simply been overlooked? I can handle the implementation, but I wanted to check before getting started.
    For multiple streams i would choose mean.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant