Enabling gitsync and PVC should work

Question

Enabling gitsync and PVC should work

ekdeveloper opened this issue a year ago · 1 comments

ekdeveloper commented a year ago

Checks

I have checked for existing issues.
This report is about the User-Community Airflow Helm Chart.

Chart Version

8.7.0

Kubernetes Version

NA

Helm Version

8.7.0

Description

airflow doc says the both gitsync and pvc can be enabled
https://airflow.apache.org/docs/helm-chart/stable/manage-dags-files.html#mounting-dags-using-git-sync-sidecar-with-persistence-enabled

https://github.com/apache/airflow/blob/97215822f90cac274e92e642fcd724cc940fe0eb/chart/templates/dag-processor/dag-processor-deployment.yaml#L135

however there is a validation check to prevent this
https://github.com/airflow-helm/charts/blob/f5c2f0cd617982c642789cf66fdef48453f84e67/charts/airflow/templates/_helpers/validate-values.tpl#LL122C1-L122C1

Relevant Logs

No response

Custom Helm Values

No response

Answer 1 · 2023-05-13T19:43:53.000Z

@ekdeveloper the docs you are referencing are for a different helm chart. This helm chart (the User-Community one) is planning to implement support for having both git-sync and PVC persistence soon (see #283), but this change has not yet been merged, hence the values-validation which prevents it for now.