set clickhouse max connections (#1175)

sentry-kubernetes · Mar 30, 2024 · d53e958 · d53e958
1 parent f4d2f74
commit d53e958
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 1 deletion.
diff --git a/sentry/Chart.yaml b/sentry/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: sentry
 description: A Helm chart for Kubernetes
 type: application
-version: 21.6.1
+version: 21.6.2
 appVersion: 24.2.0
 dependencies:
   - name: memcached

diff --git a/sentry/templates/_helper.tpl b/sentry/templates/_helper.tpl
@@ -450,6 +450,8 @@ Common Snuba environment variables
       name: {{ .Values.externalClickhouse.existingSecret }}
       key: {{ default "clickhouse-password" .Values.externalClickhouse.existingSecretKey }}
 {{- end }}
+- name: CLICKHOUSE_MAX_CONNECTIONS
+  value: {{ .Values.snuba.clickhouse.maxConnections | quote }}
 {{- end -}}
 
 {{- define "vroom.env" -}}

diff --git a/sentry/templates/configmap-snuba.yaml b/sentry/templates/configmap-snuba.yaml
@@ -25,6 +25,7 @@ data:
         "port": int({{ include "sentry.clickhouse.port" . }}),
         "user":  env("CLICKHOUSE_USER", "default"),
         "password": env("CLICKHOUSE_PASSWORD", ""),
+        "max_connections": int(os.environ.get("CLICKHOUSE_MAX_CONNECTIONS", 100)),
         "database": env("CLICKHOUSE_DATABASE", "default"),
         "http_port": {{ include "sentry.clickhouse.http_port" . }},
         "storage_sets": {

diff --git a/sentry/values.yaml b/sentry/values.yaml
@@ -953,6 +953,8 @@ snuba:
   migrateJob:
     env: []
 
+  clickhouse:
+    maxConnections: 100
 hooks:
   enabled: true
   removeOnSuccess: true