[GitHub] [airflow] thesuperzapper commented on issue #16684: Helm Chart Allow for Empty data settings

[GitBox <gi...@apache.org>]

thesuperzapper commented on issue #16684:
URL: https://github.com/apache/airflow/issues/16684#issuecomment-901529141


   Those watching might be interested in how the ["user community" chart](https://github.com/airflow-helm/charts/tree/main/charts/airflow) (not the "official" one) manages database connection strings.
   
   The "user community" chart exposes configs for "parts" of the connection string, rather than making the user provide the entire string. Most importantly, the user only has to define a Kubernetes Secret which contains their database password with the [`externalDatabase.passwordSecret`](https://github.com/airflow-helm/charts/blob/3721cfd433b5a210361def6afd63b725f9ac4044/charts/airflow/values.yaml#L1627-L1629) value. You can see the other `externalDatabase.*` values [here](https://github.com/airflow-helm/charts/blob/3721cfd433b5a210361def6afd63b725f9ac4044/charts/airflow/values.yaml#L1602-L1641).
   
   This is enabled by internally using the [`_CMD` variables](https://github.com/airflow-helm/charts/blob/3721cfd433b5a210361def6afd63b725f9ac4044/charts/airflow/templates/config/secret-config-envs.yaml#L17-L64) so the "password" secret can be used, even though we don't know its value at chart install time.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org