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

Instructions assume you have speedtest-cli installed #20

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

fermelone
Copy link

Realized after installing on a clean raspbian image that the instructions don't include the installation of speedtest-cli.

A quick installation via pip, even on runtime, fixed it.

@fmdlc
Copy link
Owner

fmdlc commented Oct 20, 2020

Hi @fermelone Thanks for submitting this PR. Speedtest-cli is installed in the Telegraf DockerImage. But if you want to run it from the bare metal it's ok to install it.

Copy link

@fdelacruz-asapp fdelacruz-asapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to add:

If you wish to run it in the baremetal, so you need to have speedtest-cli installed.

@fmdlc fmdlc force-pushed the master branch 4 times, most recently from 0c8df66 to 9eaf212 Compare October 24, 2020 21:30
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

Successfully merging this pull request may close these issues.

None yet

3 participants