Remove duplicated volume mount for git sync in scheduler template (ap…

…ache#12717) closes: apache#12716
huxuan · Feb 8, 2021 · e7a2e35 · e7a2e35
1 parent 8c5594b
commit e7a2e35
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/chart/templates/dags-persistent-volume-claim.yaml b/chart/templates/dags-persistent-volume-claim.yaml
@@ -19,7 +19,7 @@
 kind: PersistentVolumeClaim
 apiVersion: v1
 metadata:
-  name: {{ .Release.Name }}-dags
+  name: {{ template "airflow_dags_volume_claim" . }}
   labels:
     tier: airflow
     component: dags-pvc

diff --git a/chart/templates/scheduler/scheduler-deployment.yaml b/chart/templates/scheduler/scheduler-deployment.yaml
@@ -168,8 +168,6 @@ spec:
 {{ toYaml .Values.scheduler.extraVolumeMounts | indent 12 }}
 {{- end }}
 {{- if .Values.dags.gitSync.enabled }}
-            - name: dags
-              mountPath: {{ template "airflow_dags_mount_path" . }}
         {{- include "git_sync_container" . | indent 8 }}
 {{- end }}
         # Always start the garbage collector sidecar.

diff --git a/chart/values.yaml b/chart/values.yaml
@@ -757,9 +757,9 @@ dags:
     # If using a custom storageClass, pass name here
     storageClassName:
     # access mode of the persistent volume
-    accessMode: ReadWriteMany
+    accessMode: ReadWriteOnce
     ## the name of an existing PVC to use
-    existingClaim: ~
+    existingClaim:
   gitSync:
     enabled: false
     # git repo clone url