You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "roc chan (JIRA)" <ji...@apache.org> on 2018/06/13 14:47:00 UTC
[jira] [Created] (AIRFLOW-2611) Fix wrong dag volume mount path for
kubernetes executor
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)