Merge pull request #184 from langchain-ai/angus/create-clickhouse-clu…

…ster-variable

feat: Rename cluster var and add to docs

charts/langsmith/README.md

@@ -85,7 +85,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | clickhouse.config.allowSimdjson | bool | `true` |  |
 | clickhouse.containerHttpPort | int | `8123` |  |
 | clickhouse.containerNativePort | int | `9000` |  |
-| clickhouse.external.clickhouseCluster | string | `""` |  |
+| clickhouse.external.cluster | string | `""` |  |
 | clickhouse.external.database | string | `"default"` |  |
 | clickhouse.external.enabled | bool | `false` |  |
 | clickhouse.external.existingSecretName | string | `""` |  |
@@ -344,7 +344,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | clickhouse.config.allowSimdjson | bool | `true` |  |
 | clickhouse.containerHttpPort | int | `8123` |  |
 | clickhouse.containerNativePort | int | `9000` |  |
-| clickhouse.external.clickhouseCluster | string | `""` |  |
+| clickhouse.external.cluster | string | `""` |  |
 | clickhouse.external.database | string | `"default"` |  |
 | clickhouse.external.enabled | bool | `false` |  |
 | clickhouse.external.existingSecretName | string | `""` |  |

charts/langsmith/templates/_helpers.tpl

@@ -201,7 +201,7 @@ Template containing common environment variables that are used by several servic
       name: {{ include "langsmith.clickhouseSecretsName" . }}
       key: clickhouse_tls
 - name: CLICKHOUSE_CLUSTER
-  value: {{ .Values.clickhouse.external.clickhouseCluster }}
+  value: {{ .Values.clickhouse.external.cluster }}
 - name: LOG_LEVEL
   value: {{ .Values.config.logLevel }}
 {{- if .Values.config.oauth.enabled }}

charts/langsmith/values.yaml

@@ -399,7 +399,7 @@ clickhouse:
     password: "password"
     database: "default"
     tls: false
-    clickhouseCluster: ""
+    cluster: ""
     existingSecretName: ""
   containerHttpPort: 8123
   containerNativePort: 9000