[ 
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)

Reply via email to