lux4rd0 / weatherflow-dashboards-aio

An All-In-One WeatherFlow logging and dashboarding project.
https://labs.lux4rd0.com/weatherflow-collector/
GNU Affero General Public License v3.0
68 stars 11 forks source link

Environment variables not being read by generate_docker-compose.sh #1

Open thebaldgeek opened 3 years ago

thebaldgeek commented 3 years ago

Is there any way to edit generate_docker-compose.sh with the require variables?

I have set the environment variables as directed. When I read them back $ echo $WEATHERFLOW_COLLECTOR_THREADS 4 they are set. When I read them back as set | less I can see them:

TERM=xterm
UID=1000
USER=beno
WEATHERFLOW_COLLECTOR_IMPORT_DAYS=365
WEATHERFLOW_COLLECTOR_THREADS=4
WEATHERFLOW_COLLECTOR_TOKEN=aaaaaa56-bbbb-43e6-9e28-21bbac7c86d4

Regardless, the generate_docker-compose.sh returns the error:

./generate_docker-compose.sh

import_days=
loki_client_url=
perf_interval=
threads=
token=
WEATHERFLOW_COLLECTOR_IMPORT_DAYS variable not set. Defaulting to 365 days
WEATHERFLOW_COLLECTOR_LOKI_CLIENT_URL was not set. No default being set
WEATHERFLOW_COLLECTOR_THREADS was not set. Setting defaults: 4
WEATHERFLOW_COLLECTOR_PERF_INTERVAL environmental variable not set. Defaulting to 60 seconds
Missing authentication token. Please provide your token as a command parameter.

Same result with 'bash ./generate_docker-compose.sh'

Can I edit the generate_docker-compose.sh script to have the required variables?

Thanks for your time.

lux4rd0 commented 3 years ago

Hello there! Yeah - I saw that I left a little line that pushes the API token back into the script.

I did release a brand new version just now... That simplifies things a fair amount. Can you try downloading v3.2.2 and see if that helps out?

Also - your token that you have listed - is that your real token? When I try to see what happens with it says "unauthorized," - so either it's not a real key or something else is weird.

thebaldgeek commented 3 years ago

Far out! This is amazing!
You have clearly spent a lot of time on this since I last downloaded it and got it running.
Major good job and thanks so much for sharing.

So yes, the new version had the fix I needed, followed the readme.md and it all just worked (tweaked the versions on all commands of course). Everything got pulled down and it just came up seamlessly.
Thanks for the update/fix and I am looking forward to seeing the data fill out (I have an original Sky and Air, and some data is not showing up yet, but I think it did this last time I ran the docker).

lux4rd0 commented 3 years ago

Here's a quick snapshot of what I'm testing against - it's a mix of Tempest, Skys, and Airs - depending on how they're connected to a hub as I group by station. But yeah - if you want to share a key I can test it with you. I've been fortunate to have a few more Tempest users test out the AIO and they had it showing data in a few minutes. However - it's always a fun way to test things out. Also - if there are updates to the steps, please let me know and I'll update the docs to reflect a FAQ. :)

multi-current_conditions