roc chan created AIRFLOW-2611:
---------------------------------
Summary: Fix wrong dag volume mount path for kubernetes executor
Key: AIRFLOW-2611
URL: https://issues.apache.org/jira/browse/AIRFLOW-2611
Project: Apache Airflow
Issue Type: Bug
Components: contrib
Affects Versions: 2.0.0, 1.10
Reporter: roc chan
Assignee: roc chan
If I set both *dags_volume_claim* and *git_subpath*, well, I won't get the
volume mount path that I want ([see
code|https://github.com/apache/incubator-airflow/blob/0f4d681f6f6e15acd1399dede146e75cb688d536/airflow/contrib/kubernetes/worker_configuration.py#L139]):
{code}
volume_mounts = [{
'name': dags_volume_name,
'mountPath': os.path.join(
self.worker_airflow_dags,
self.kube_config.git_subpath
),
'readOnly': True
}, {
'name': logs_volume_name,
'mountPath': self.worker_airflow_logs
}]
{code}
There are two way of syncing dags, *pvc* and *git-sync*, I think if set both,
the priority of *pvc* should higher than *git-sync*, because some beginers use
the template config file at the beginning ([like
this|https://github.com/apache/incubator-airflow/blob/0f4d681f6f6e15acd1399dede146e75cb688d536/airflow/contrib/kubernetes/worker_configuration.py#L139]),
when they want to use pvc to mount dags, they may don't know they should
remove the *git_subpath* config.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
