search

finaldie / auto-news

A personal news aggregator to pull information from multi-sources + LLM (ChatGPT/Gemini/Ollama via LangChain) to help us reading efficiently with less noises, the sources including: Tweets, RSS, YouTube, Web Articles, Reddit, and personal Journal notes.
MIT License
519 stars 56 forks source link

TZ not available in compose file as well as few PROJ dir #87

Open flowcool opened 3 months ago

flowcool commented 3 months ago

Hello,

I believe we can had TZ as variable in env and compose file.

I noticed also a few env variable not declared any where. Personally I added them into my env. file:

AIRFLOW_PROJ_DIR=
POSTGRES_PROJ_DIR=
REDIS_PROJ_DIR=
MYSQL_PROJ_DIR=
MILVUS_PROJ_DIR=
finaldie commented 3 months ago

Hi @flowcool

The above vars are only for docker installation which is defined in the install.env. Since the system is currently using the UTC timezone, may I learn what's the use case for the TZ var? Thanks.

flowcool commented 3 months ago

Hello,

Thanks for clarification of install.env; I might have missed that.

I personally use TZ in all setup to have time adjusted to my Timezone, easier to troubleshoot when we're looking at the logs (what I'm doing a lot on this project). It's clearly not a must but probably more a best practice.

I can share my conf if you want to have a look

finaldie commented 3 months ago

Cool, thanks for sharing. Yes, I would appreciate it if you could share your configuration so I can learn from it.

I used that timezone switch button from Airflow to toggle the UTC and local time. image

flowcool commented 3 months ago

Please find compose attached and env file needs to have: TZ=Europe/Paris (Anything you want)

For the compose, I added TZ for DBs too.

Cheers, docker-compose.yaml.txt