SPEEDTEST_SPEEDTEST_INTERVAL variable should be SPEEDTEST_INTERVAL?

frdmn / docker-speedtest-grafana

🚢📦Grafana + InfluxDB + Speedtest.net results

MIT License

206 stars 71 forks source link

SPEEDTEST_SPEEDTEST_INTERVAL variable should be SPEEDTEST_INTERVAL? #14

Open n1nj4888 opened 4 years ago

n1nj4888 commented 4 years ago

Hi There,

The README lists one of the environment variables as SPEEDTEST_SPEEDTEST_INTERVAL but this is actually referenced in the code as SPEEDTEST_INTERVAL. the .env.sample and docker-compose.yml also reference this as SPEEDTEST_SPEEDTEST_INTERVAL and $SPEEDTEST_SPEEDTEST_INTERVAL but this should just be SPEEDTEST_INTERVAL?

Thanks!

frdmn commented 4 years ago

It should be SPEEDTEST_INTERVAL in the README. I was using some {{CONTAINER}}_{{KEY}} format for environment keys in the .env file some time ago.

n1nj4888 commented 4 years ago

The pull request I raised for "enhanced-logging" also fixed this README issue.