Revert "Add CLICKHOUSE_PORT as option for externalClickhouse configs (#…

…415)" (#417) This reverts commit 6cc47a0.
PostHog · Jun 2, 2022 · 484f8f5 · 484f8f5
1 parent fbbbd14
commit 484f8f5
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 10 deletions.
diff --git a/charts/posthog/templates/_clickhouse.tpl b/charts/posthog/templates/_clickhouse.tpl
@@ -18,8 +18,6 @@
 {{- else -}}
 - name: CLICKHOUSE_HOST
   value: {{ required "externalClickhouse.host is required if not clickhouse.enabled" .Values.externalClickhouse.host | quote }}
-- name: CLICKHOUSE_PORT
-  value: {{ .Values.externalClickhouse.port | quote }}
 - name: CLICKHOUSE_CLUSTER
   value: {{ required "externalClickhouse.cluster is required if not clickhouse.enabled" .Values.externalClickhouse.cluster | quote }}
 - name: CLICKHOUSE_DATABASE

diff --git a/charts/posthog/tests/__snapshot__/_clickhouse.tpl.yaml.snap b/charts/posthog/tests/__snapshot__/_clickhouse.tpl.yaml.snap
@@ -34,8 +34,6 @@ should render with external clickhouse:
   1: |
     - name: CLICKHOUSE_HOST
       value: foo.bar.net
-    - name: CLICKHOUSE_PORT
-      value: "9000"
     - name: CLICKHOUSE_CLUSTER
       value: somecluster
     - name: CLICKHOUSE_DATABASE
@@ -52,8 +50,6 @@ should render with external clickhouse with more custom settings:
   1: |
     - name: CLICKHOUSE_HOST
       value: foo.bar.net
-    - name: CLICKHOUSE_PORT
-      value: "9000"
     - name: CLICKHOUSE_CLUSTER
       value: customCluster
     - name: CLICKHOUSE_DATABASE
@@ -70,8 +66,6 @@ should render with external clickhouse with secrets:
   1: |
     - name: CLICKHOUSE_HOST
       value: foo.bar.net
-    - name: CLICKHOUSE_PORT
-      value: "9000"
     - name: CLICKHOUSE_CLUSTER
       value: somecluster
     - name: CLICKHOUSE_DATABASE

diff --git a/charts/posthog/values.yaml b/charts/posthog/values.yaml
@@ -737,8 +737,6 @@ clickhouse:
 externalClickhouse:
   # -- Host of the external cluster. This is required when clickhouse.enabled is false
   host:
-  # -- Port of the external cluster. 
-  port: 9000
   # -- Name of the external cluster to run DDL queries on. This is required when clickhouse.enabled is false
   cluster:
   # -- Database name for the external cluster