Merge pull request #18 from langchain-ai/infra/allow-disabling-migrat…

…ions Allow disabling migrations
langchain-ai · Nov 27, 2023 · 8cb5fc8 · 8cb5fc8
2 parents a3f9ce0 + c011795
commit 8cb5fc8
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 19 deletions.
diff --git a/charts/langsmith/Chart.yaml b/charts/langsmith/Chart.yaml
@@ -5,5 +5,5 @@ maintainers:
     email: [email protected]
 description: Helm chart to deploy the langsmith application and all services it depends on.
 type: application
-version: 0.1.4
+version: 0.1.5
 appVersion: "0.1.0"
diff --git a/charts/langsmith/templates/_helpers.tpl b/charts/langsmith/templates/_helpers.tpl
@@ -125,9 +125,6 @@ Template containing common environment variables that are used by several servic
 - name: AUTH_TYPE
 {{- if .Values.config.oauth.enabled }}
   value: "oauth"
-{{- else }}
-  value: "none"
-{{- end }}
 - name: OAUTH_CLIENT_ID
   valueFrom:
     secretKeyRef:
@@ -138,19 +135,14 @@ Template containing common environment variables that are used by several servic
     secretKeyRef:
       name: {{ include "langsmith.secretsName" . }}
       key: oauth_issuer_url
+{{- else }}
+  value: "none"
+{{- end }}
 - name: LANGSMITH_LICENSE_KEY
   valueFrom:
     secretKeyRef:
       name: {{ include "langsmith.secretsName" . }}
       key: langsmith_license_key
-- name: CLICKHOUSE_HOST
-  value: "clickhouse"
-- name: CLICKHOUSE_USER
-  value: clickhouse_admin
-- name: CLICKHOUSE_PASSWORD
-  value: password
-- name: CLICKHOUSE_DB
-  value: default
 - name: API_KEY_SALT
   valueFrom:
     secretKeyRef:

diff --git a/charts/langsmith/templates/backend/run-migrations.yaml b/charts/langsmith/templates/backend/run-migrations.yaml
@@ -1,3 +1,4 @@
+{{- if .Values.backend.migrations.enabled }}
 apiVersion: batch/v1
 kind: Job
 metadata:
@@ -59,3 +60,4 @@ spec:
       tolerations:
         {{- toYaml . | nindent 8 }}
       {{- end }}
+  {{- end }}
diff --git a/charts/langsmith/templates/frontend/deployment.yaml b/charts/langsmith/templates/frontend/deployment.yaml
@@ -46,13 +46,6 @@ spec:
             - name: VITE_BACKEND_AUTH_TYPE
             {{- if .Values.config.oauth.enabled }}
               value: "oauth"
-            {{- else }}
-              value: "none"
-            {{- end }}
-            - name: VITE_BACKEND_URL
-              value: ""
-            - name: VITE_HUB_WEB_URL
-              value: ""
             - name: VITE_OAUTH_CLIENT_ID
               valueFrom:
                 secretKeyRef:
@@ -63,6 +56,13 @@ spec:
                 secretKeyRef:
                   key: oauth_issuer_url
                   name: {{ include "langsmith.secretsName" .}}
+            {{- else }}
+              value: "none"
+            {{- end }}
+            - name: VITE_BACKEND_URL
+              value: ""
+            - name: VITE_HUB_WEB_URL
+              value: ""
             {{- with .Values.frontend.deployment.extraEnv }}
               {{- toYaml . | nindent 12 }}
             {{- end }}

diff --git a/charts/langsmith/values.yaml b/charts/langsmith/values.yaml
@@ -75,6 +75,7 @@ backend:
     tolerations: []
     affinity: {}
   migrations:
+    enabled: true
     replicaCount: 1
     labels: {}
     annotations: {}