[
https://issues.apache.org/jira/browse/AIRFLOW-2611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kaxil Naik resolved AIRFLOW-2611.
---------------------------------
Resolution: Fixed
Fix Version/s: 2.0.0
1.10.0
Issue resolved by pull request #3497
[https://github.com/apache/incubator-airflow/pull/3497]
> 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
> Priority: Major
> Fix For: 1.10.0, 2.0.0
>
>
> 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/scripts/ci/kubernetes/kube/configmaps.yaml#L184]),
> when they want to use pvc to mount dags, they may don't know they should
> remove the *git_subpath* config. So I think when *dags_volume_claim* is set,
> the *mountPath* shuold not join the *git_subpath*
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)