Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Results wrong: download speed 10x high #25

Closed
robbat2 opened this issue May 12, 2024 · 4 comments
Closed

Results wrong: download speed 10x high #25

robbat2 opened this issue May 12, 2024 · 4 comments

Comments

@robbat2
Copy link

robbat2 commented May 12, 2024

There's a performance bug somewhere: this is reading my download speed as 10x too high compared to speedtest-cli

exporter:

# HELP promhttp_metric_handler_errors_total Total number of internal errors encountered by the promhttp metric handler.
# TYPE promhttp_metric_handler_errors_total counter
promhttp_metric_handler_errors_total{cause="encoding"} 0
promhttp_metric_handler_errors_total{cause="gathering"} 0
# HELP speedtest_data_used_megabytes Data used for speedtest in MB
# TYPE speedtest_data_used_megabytes gauge
speedtest_data_used_megabytes{ip="REDACTED",isp="REDACTED"} 6540.296778
# HELP speedtest_download_megabits_per_second Speedtest current Download Speed in Mbit/s
# TYPE speedtest_download_megabits_per_second gauge
speedtest_download_megabits_per_second{ip="REDACTED",isp="REDACTED"} 6610.954661656727
# HELP speedtest_jitter_latency_milliseconds Speedtest current Jitter in ms
# TYPE speedtest_jitter_latency_milliseconds gauge
speedtest_jitter_latency_milliseconds{ip="REDACTED",isp="REDACTED"} 0
# HELP speedtest_ping_latency_milliseconds Speedtest current Ping in ms
# TYPE speedtest_ping_latency_milliseconds gauge
speedtest_ping_latency_milliseconds{ip="REDACTED",isp="REDACTED"} 0
# HELP speedtest_up Indicates if the speedtest was successful
# TYPE speedtest_up gauge
speedtest_up 1
# HELP speedtest_upload_megabits_per_second Speedtest current Upload Speed in Mbit/s
# TYPE speedtest_upload_megabits_per_second gauge
speedtest_upload_megabits_per_second{ip="REDACTED",isp="REDACTED"} 299.8439327369734

speedtest-cli:

# speedtest-cli 
Retrieving speedtest.net configuration...
Testing from REDACTED...
Retrieving speedtest.net server list...
Selecting best server based on ping...
Hosted by REDACTED: 6.049 ms
Testing download speed................................................................................
Download: 318.80 Mbit/s
Testing upload speed......................................................................................................
Upload: 673.73 Mbit/s
@heathcliff26
Copy link
Owner

Are you using the cli version or the slim version?

@robbat2
Copy link
Author

robbat2 commented May 12, 2024

This was the docker.io/heathcliff26/speedtest-exporter container, using go-native implementation, not the binary.

I switched to ghcr.io/miguelndecarvalho/speedtest-exporter for now, it's good enough

@heathcliff26
Copy link
Owner

If it works for you, that is fine. Alternatively you could also use the cli tag, that also call speedtest-cli under the hood, same as miguelndecarvalho/speedtest-exporter

@heathcliff26
Copy link
Owner

This seems to have been a bug in the go-speedtest library i use. It has been fixed upstream and the dependency is also update here, so i'm closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants