-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Speedtest command appears to be broken #47
Comments
Command speedtest | egrep... work correctly in console:
But speedtest-cli |egrep... have result beautefull:
But I don't see this command being executed from speedtest.py. The process with "speedtest" doesn't appear.
My WLANPI:
|
Thanks for installing the binary and doing this testing (#46). It's clear there are performance differences in speedtest-cli and speedtest from Ookla in your results posted above. Possibly something changed in the Ookla binary between when Nigel wrote his wrapper code 3 years ago. Needs further attention from someone. |
Do you see still see |
I found error. speedtest need to start with parameters:
I will made pull request. Please check. Additional information from /var/log/syslog for this solution:
speedtest want to write info in to /home/wlanpi-chat-bot/.config/ookla/speedtest-cli.json but wlanpi not contains path /home/wlanpi-chat-bot
But speedtest and speedtest-cli have different result:
|
As mentioned in PR #49, we absolutely cannot run with |
Please show me page in guide of wlanpi-chat-bot about information about license agreement of speedtest. I not found this information in guide. Therefore, users don't know that this needs to be done. And also don't know that for the chat bot you need to create the home directory in home. All this leads to the fact that using speedtest is not possible. You can refer to various agreements for a long time, but make the solution work. |
Ookla speedtest is not a WLAN Pi application. Thus, WLAN Pi cannot automatically accept the EULA on behalf of the user. Add changes to incorporate a mechanism for the user to accept the EULA. |
The approach that I would approve in a PR would be the command to return an error if the EULA is not accepted, then instruct the user to review the EULA and run a separate command to accept the EULA which would also handle any other setup like having a home directory. |
You can describe all the necessary actions in your documentation. This may be a working solution. |
Okay. My version of solution. Will this work for you? Wlanpi Chat Bot work on user wlanpi-chat-bot.
User must create home directory for user wlanpi-chat-bot and other directory:
User must copy file speedtest-cli.json to /home/wlanpi-chat-bot/.config/ookla
User must change owner for /home/wlanpi-chat-bot and other directory:
User must restart service whanpi-chat-bot:
End. We no longer see errors when running speedtest from Telegram. What do you say? If yes, I can add it to the documentation tomorrow. |
License accepted:
I don't have process after command "speedtest" in bot:
Originally posted by @gcastle2007 in #46 (comment)
The text was updated successfully, but these errors were encountered: