Fix: Allow clickhouse data and logs volumes to have different file sy…

…stems (#74)
datafold · Jun 2, 2024 · 89e0016 · 89e0016
1 parent 41b628f
commit 89e0016
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/charts/datafold/Chart.yaml b/charts/datafold/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: datafold
 description: Helm chart package to deploy Datafold on kubernetes.
 type: application
-version: 0.6.2
+version: 0.6.3
 appVersion: "1.0.0"
 icon: https://www.datafold.com/logo.png
 

diff --git a/charts/datafold/charts/clickhouse/templates/persistent-volume.yaml b/charts/datafold/charts/clickhouse/templates/persistent-volume.yaml
@@ -18,13 +18,13 @@ spec:
   csi:
     driver: ebs.csi.aws.com
     volumeHandle: {{ .Values.storage.dataVolumeId }}
-    fsType: "ext4"
+    fsType: {{ .Values.storage.dataFsType }}
 {{- end }}
 {{- if (eq .Values.global.cloudProvider "gcp") }}
   csi:
     driver: pd.csi.storage.gke.io
     volumeHandle: {{ .Values.storage.dataVolumeId }}
-    fsType: "ext4"
+    fsType: {{ .Values.storage.dataFsType }}
 {{- end }}
   claimRef:
     name: {{ include "clickhouse.data.pvc.name" . }}
@@ -51,13 +51,13 @@ spec:
   csi:
     driver: ebs.csi.aws.com
     volumeHandle: {{ .Values.storage.logVolumeId }}
-    fsType: "ext4"
+    fsType: {{ .Values.storage.logsFsType }}
 {{- end }}
 {{- if (eq .Values.global.cloudProvider "gcp") }}
   csi:
     driver: pd.csi.storage.gke.io
     volumeHandle: {{ .Values.storage.logVolumeId }}
-    fsType: "ext4"
+    fsType: {{ .Values.storage.logsFsType }}
 {{- end }}
   claimRef:
     name: {{ include "clickhouse.logs.pvc.name" . }}

diff --git a/charts/datafold/charts/clickhouse/values.yaml b/charts/datafold/charts/clickhouse/values.yaml
@@ -18,6 +18,8 @@ storage:
   storageClass: ""
   dataVolumeId: ""
   logVolumeId: ""
+  dataFsType: "ext4"
+  logsFsType: "ext4"
 
 config:
   ram_ratio: 0.9

diff --git a/charts/datafold/values.yaml b/charts/datafold/values.yaml
@@ -96,7 +96,7 @@ config:
   enableSsrfProtection: "true"
   logLevel: "INFO"
   enforceHttps: "true"
-  googleOauthRequestBQ: "true"
+  googleOauthRequestBQ: "false"
   ddTraceEnabled: "false"
   ddApmAddress: ""
   ddDogStatsDisable: "false"