Skip to content

Commit 4813756

Browse files
committed
fix webapp and supervisor templates
1 parent 2b9dbf1 commit 4813756

File tree

5 files changed

+43
-13
lines changed

5 files changed

+43
-13
lines changed

hosting/k8s/helm/templates/_helpers.tpl

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -166,13 +166,30 @@ ClickHouse hostname
166166
{{- end }}
167167

168168
{{/*
169-
ClickHouse URL
169+
ClickHouse URL for application (with secure parameter)
170170
*/}}
171171
{{- define "trigger-v4.clickhouse.url" -}}
172-
{{- if .Values.clickhouse.host -}}
173-
http://{{ .Values.clickhouse.username }}:{{ .Values.clickhouse.password }}@{{ .Values.clickhouse.host }}:{{ .Values.clickhouse.httpPort | default 8123 }}
174-
{{- else if .Values.clickhouse.deploy -}}
175-
http://{{ .Values.clickhouse.auth.username }}:{{ .Values.clickhouse.auth.password }}@{{ include "trigger-v4.clickhouse.hostname" . }}:8123
172+
{{- if .Values.clickhouse.deploy -}}
173+
{{- $protocol := ternary "https" "http" .Values.clickhouse.secure -}}
174+
{{- $secure := ternary "true" "false" .Values.clickhouse.secure -}}
175+
{{ $protocol }}://{{ .Values.clickhouse.auth.username }}:{{ .Values.clickhouse.auth.password }}@{{ include "trigger-v4.clickhouse.hostname" . }}:8123?secure={{ $secure }}
176+
{{- else if .Values.clickhouse.external.host -}}
177+
{{- $protocol := ternary "https" "http" .Values.clickhouse.external.secure -}}
178+
{{- $secure := ternary "true" "false" .Values.clickhouse.external.secure -}}
179+
{{ $protocol }}://{{ .Values.clickhouse.external.username }}:{{ .Values.clickhouse.external.password }}@{{ .Values.clickhouse.external.host }}:{{ .Values.clickhouse.external.httpPort | default 8123 }}?secure={{ $secure }}
180+
{{- end -}}
181+
{{- end }}
182+
183+
{{/*
184+
ClickHouse URL for replication (without secure parameter)
185+
*/}}
186+
{{- define "trigger-v4.clickhouse.replication.url" -}}
187+
{{- if .Values.clickhouse.deploy -}}
188+
{{- $protocol := ternary "https" "http" .Values.clickhouse.secure -}}
189+
{{ $protocol }}://{{ .Values.clickhouse.auth.username }}:{{ .Values.clickhouse.auth.password }}@{{ include "trigger-v4.clickhouse.hostname" . }}:8123
190+
{{- else if .Values.clickhouse.external.host -}}
191+
{{- $protocol := ternary "https" "http" .Values.clickhouse.external.secure -}}
192+
{{ $protocol }}://{{ .Values.clickhouse.external.username }}:{{ .Values.clickhouse.external.password }}@{{ .Values.clickhouse.external.host }}:{{ .Values.clickhouse.external.httpPort | default 8123 }}
176193
{{- end -}}
177194
{{- end }}
178195

hosting/k8s/helm/templates/supervisor.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
{{- if .Values.supervisor.deploy }}
21
{{- if .Values.supervisor.serviceAccount.create }}
32
apiVersion: v1
43
kind: ServiceAccount
@@ -282,5 +281,4 @@ spec:
282281
protocol: TCP
283282
name: metrics
284283
selector:
285-
{{- include "trigger-v4.componentSelectorLabels" (dict "Chart" .Chart "Release" .Release "Values" .Values "component" $component) | nindent 4 }}
286-
{{- end }}
284+
{{- include "trigger-v4.componentSelectorLabels" (dict "Chart" .Chart "Release" .Release "Values" .Values "component" $component) | nindent 4 }}

hosting/k8s/helm/templates/webapp.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ spec:
224224
- name: RUN_REPLICATION_ENABLED
225225
value: "1"
226226
- name: RUN_REPLICATION_CLICKHOUSE_URL
227-
value: {{ include "trigger-v4.clickhouse.url" . | quote }}
227+
value: {{ include "trigger-v4.clickhouse.replication.url" . | quote }}
228228
- name: RUN_REPLICATION_LOG_LEVEL
229229
value: {{ .Values.webapp.runReplication.logLevel | quote }}
230230
{{- if not .Values.telemetry.enabled }}

hosting/k8s/helm/values-production-example.yaml

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,8 @@ redis:
7979

8080
# Production ClickHouse
8181
clickhouse:
82+
# Set to true to enable TLS/secure connections in production
83+
secure: true
8284
persistence:
8385
enabled: true
8486
size: 100Gi
@@ -123,8 +125,7 @@ supervisor:
123125
memory: 512Mi
124126
# Example: Use external PostgreSQL instead
125127
# postgres:
126-
# enabled: false
127-
# external: true
128+
# deploy: false
128129
# external:
129130
# host: "your-postgres-host.rds.amazonaws.com"
130131
# port: 5432
@@ -136,9 +137,19 @@ supervisor:
136137

137138
# Example: Use external Redis instead
138139
# redis:
139-
# enabled: false
140-
# external: true
140+
# deploy: false
141141
# external:
142142
# host: "your-redis-cluster.cache.amazonaws.com"
143143
# port: 6379
144144
# password: "your-redis-password"
145+
146+
# Example: Use external ClickHouse instead
147+
# clickhouse:
148+
# deploy: false
149+
# external:
150+
# host: "your-clickhouse-host.cloud.provider.com"
151+
# httpPort: 8443 # Use 8443 for HTTPS
152+
# nativePort: 9440 # Use 9440 for secure native connections
153+
# username: "trigger_user"
154+
# password: "your-clickhouse-password"
155+
# secure: true # Use true for TLS/secure connections

hosting/k8s/helm/values.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -413,6 +413,9 @@ electric:
413413
clickhouse:
414414
deploy: true
415415

416+
# TLS/Secure connection configuration
417+
secure: false # Set to true to use HTTPS and secure connections
418+
416419
# Bitnami ClickHouse chart configuration (when deploy: true)
417420
auth:
418421
username: "default"
@@ -437,6 +440,7 @@ clickhouse:
437440
nativePort: 9000
438441
username: ""
439442
password: ""
443+
secure: false # Set to true for external secure connections
440444

441445
# ClickHouse configuration override
442446
# These defaults are based on official recommendations for systems with <16GB RAM:

0 commit comments

Comments
 (0)