search

airflow-helm / charts

The User-Community Airflow Helm Chart is the standard way to deploy Apache Airflow on Kubernetes with Helm. Originally created in 2017, it has since helped thousands of companies create production-ready deployments of Airflow on Kubernetes.
https://github.com/airflow-helm/charts/tree/main/charts/airflow
Apache License 2.0
665 stars 476 forks source link

Add TZ option #817

Closed achovelon closed 7 months ago

achovelon commented 10 months ago

Checks

Motivation

Hello, I would like to have the option to choose my default TZ for airflow scheduler. On the chart it's fixed to etc/UTC : https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/config/secret-config-envs.yaml#L15 Do you think it's a good idea to variabilize this option? Or to just remove it from this secret so we could configure it using the "config" section. Thanks for your help here. BR,

Implementation

If we could remove line 15 : https://github.com/airflow-helm/charts/blob/main/charts/airflow/templates/config/secret-config-envs.yaml#L15 Or add vars here.

Are you willing & able to help?

thesuperzapper commented 7 months ago

I highly recommend using UTC as your timezone, this is in line with Airflow's official recommendations.

Note, this will not stop you creating DAG schedules in any timezone you like: