fupdate

langchain-ai · Nov 14, 2024 · e75ba7c · e75ba7c
1 parent 9982d50
commit e75ba7c
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/charts/langsmith/README.md b/charts/langsmith/README.md
@@ -90,7 +90,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | clickhouse.external.enabled | bool | `false` |  |
 | clickhouse.external.existingSecretName | string | `""` |  |
 | clickhouse.external.host | string | `""` |  |
-| clickhouse.external.hosting | string | `"self-hosted"` | Either "self-hosted" or "hybrid" |
+| clickhouse.external.hybrid | bool | `false` | Set to true if using managed ClickHouse |
 | clickhouse.external.nativePort | string | `"9000"` |  |
 | clickhouse.external.password | string | `"password"` |  |
 | clickhouse.external.port | string | `"8123"` |  |
@@ -349,7 +349,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | clickhouse.external.enabled | bool | `false` |  |
 | clickhouse.external.existingSecretName | string | `""` |  |
 | clickhouse.external.host | string | `""` |  |
-| clickhouse.external.hosting | string | `"self-hosted"` | Either "self-hosted" or "hybrid" |
+| clickhouse.external.hybrid | bool | `false` | Set to true if using managed ClickHouse |
 | clickhouse.external.nativePort | string | `"9000"` |  |
 | clickhouse.external.password | string | `"password"` |  |
 | clickhouse.external.port | string | `"8123"` |  |

diff --git a/charts/langsmith/examples/blob_storage_config.yaml b/charts/langsmith/examples/blob_storage_config.yaml
@@ -22,8 +22,8 @@ config:
 clickhouse:
   external:
     enabled: true
-    # Set hosting: "hybrid" if using managed ClickHouse
-    hosting: "self-hosted"
+    # Set hybrid: true if using managed ClickHouse
+    hybrid: false
 
 # If using IRSA (only for EKS), you can specify the service account annotations here.
 #backend:

diff --git a/charts/langsmith/templates/_helpers.tpl b/charts/langsmith/templates/_helpers.tpl
@@ -279,7 +279,7 @@ Template containing common environment variables that are used by several servic
 - name: BLOB_STORAGE_ENGINE
   value: {{ .Values.config.blobStorage.engine | quote }}
 - name: MIN_BLOB_STORAGE_SIZE_KB
-  value: {{ ternary 0 .Values.config.blobStorage.minBlobStorageSizeKb (eq .Values.clickhouse.external.hosting "hybrid") | quote }}
+  value: {{ ternary 0 .Values.config.blobStorage.minBlobStorageSizeKb .Values.clickhouse.external.hybrid | quote }}
 {{- if eq .Values.config.blobStorage.engine "S3" }}
 - name: S3_BUCKET_NAME
   value: {{ .Values.config.blobStorage.bucketName | quote }}
@@ -322,7 +322,7 @@ Template containing common environment variables that are used by several servic
 {{- end }}
 {{- end }}
 - name: FF_CH_SEARCH_ENABLED
-  value: {{ ternary "false" .Values.config.blobStorage.chSearchEnabled (eq .Values.clickhouse.external.hosting "hybrid") | quote }}
+  value: {{ ternary "false" .Values.config.blobStorage.chSearchEnabled .Values.clickhouse.external.hybrid | quote }}
 {{ include "langsmith.conditionalEnvVarsResolved" . }}
 {{- end }}
 

diff --git a/charts/langsmith/values.yaml b/charts/langsmith/values.yaml
@@ -390,8 +390,8 @@ clickhouse:
     # If enabled, use the following values to connect to an external database. This will also disable the
     # creation of a clickhouse stateful-set and service.
     enabled: false
-    # -- Either "self-hosted" or "hybrid"
-    hosting: "self-hosted"
+    # -- Set to true if using managed ClickHouse
+    hybrid: false
     host: ""
     port: "8123"
     nativePort: "9000"