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

Update request: speedtest_exporter #233062

Open
bjornfor opened this issue May 20, 2023 · 5 comments
Open

Update request: speedtest_exporter #233062

bjornfor opened this issue May 20, 2023 · 5 comments
Labels
9.needs: package (update) This needs a package to be updated

Comments

@bjornfor
Copy link
Contributor

Our current pkgs.speedtest_exporter points to an archived / read-only git repo: https://github.com/nlamirault/speedtest_exporter. (Also it miscalculates bandwidth numbers, compared to speedtest-cli.)

There's a ton of forks and re-implementations of prometheus speedtest exporter functionality, but in my quick search https://github.com/caarlos0/speedtest-exporter looked like good option.

CC @matthiasbeyer, @bryanasdev000, @urandom2

@bjornfor bjornfor added the 9.needs: package (update) This needs a package to be updated label May 20, 2023
@bjornfor
Copy link
Contributor Author

(Also it miscalculates bandwidth numbers, compared to speedtest-cli.)

That was reported upstream but nothing happened: nlamirault/speedtest_exporter#55

@colemickens
Copy link
Member

Huh, it's quite disappointing that it seems abandonded. I haven't found another exporter that seems to contain as many metrics. I'd be curious if you settled on something else @bjornfor

@bjornfor
Copy link
Contributor Author

@colemickens: I'm using https://github.com/caarlos0/speedtest-exporter now. It depends on proprietary speedtest program, see caarlos0/speedtest-exporter#13. Unfortunately, the open source alternative, https://github.com/sivel/speedtest-cli, doesn't doesn't seem like a good alternative either (AFAIR, it's not well maintained and has bugs?).

@pinpox
Copy link
Member

pinpox commented Aug 23, 2023

Can this be closed, as #246320 was merged now?

@bjornfor
Copy link
Contributor Author

bjornfor commented Aug 23, 2023

If this is closed then a new issue should be made that links to this one and contains half of the OP text. (EDIT: typo.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.needs: package (update) This needs a package to be updated
Projects
None yet
Development

No branches or pull requests

3 participants