chore: support configuring ttlSeconds (#172)

* chore: support configuring ttlSeconds * chore: support configuring ttlSeconds
langchain-ai · Nov 6, 2024 · aa62e71 · aa62e71
1 parent ab29f1e
commit aa62e71
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 5 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.8.4
+version: 0.8.5
 appVersion: "0.8.18"
diff --git a/charts/langsmith/README.md b/charts/langsmith/README.md
@@ -1,6 +1,6 @@
 # langsmith
 
-![Version: 0.8.4](https://img.shields.io/badge/Version-0.8.4-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.8.18](https://img.shields.io/badge/AppVersion-0.8.18-informational?style=flat-square)
+![Version: 0.8.5](https://img.shields.io/badge/Version-0.8.5-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.8.18](https://img.shields.io/badge/AppVersion-0.8.18-informational?style=flat-square)
 
 Helm chart to deploy the langsmith application and all services it depends on.
 
@@ -223,6 +223,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | backend.authBootstrap.sidecars | list | `[]` |  |
 | backend.authBootstrap.tolerations | list | `[]` |  |
 | backend.authBootstrap.topologySpreadConstraints | list | `[]` |  |
+| backend.authBootstrap.ttlSecondsAfterFinished | int | `600` |  |
 | backend.authBootstrap.volumeMounts | list | `[]` |  |
 | backend.authBootstrap.volumes | list | `[]` |  |
 | backend.autoscaling.createHpa | bool | `true` |  |
@@ -246,6 +247,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | backend.clickhouseMigrations.sidecars | list | `[]` |  |
 | backend.clickhouseMigrations.tolerations | list | `[]` |  |
 | backend.clickhouseMigrations.topologySpreadConstraints | list | `[]` |  |
+| backend.clickhouseMigrations.ttlSecondsAfterFinished | int | `600` |  |
 | backend.clickhouseMigrations.volumeMounts | list | `[]` |  |
 | backend.clickhouseMigrations.volumes | list | `[]` |  |
 | backend.containerPort | int | `1984` |  |
@@ -313,6 +315,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 | backend.migrations.sidecars | list | `[]` |  |
 | backend.migrations.tolerations | list | `[]` |  |
 | backend.migrations.topologySpreadConstraints | list | `[]` |  |
+| backend.migrations.ttlSecondsAfterFinished | int | `600` |  |
 | backend.migrations.volumeMounts | list | `[]` |  |
 | backend.migrations.volumes | list | `[]` |  |
 | backend.name | string | `"backend"` |  |

diff --git a/charts/langsmith/templates/backend/auth-bootstrap.yaml b/charts/langsmith/templates/backend/auth-bootstrap.yaml
@@ -29,7 +29,7 @@ metadata:
     {{- end }}
 spec:
   backoffLimit: 3
-  ttlSecondsAfterFinished: 600
+  ttlSecondsAfterFinished: {{ .Values.backend.authBootstrap.ttlSecondsAfterFinished }}
   template:
     metadata:
       {{- with .Values.backend.authBootstrap.annotations }}

diff --git a/charts/langsmith/templates/backend/clickhouse-migrations.yaml b/charts/langsmith/templates/backend/clickhouse-migrations.yaml
@@ -20,7 +20,7 @@ metadata:
     {{- end }}
 spec:
   backoffLimit: 2
-  ttlSecondsAfterFinished: 600
+  ttlSecondsAfterFinished: {{ .Values.backend.clickhouseMigrations.ttlSecondsAfterFinished }}
   template:
     metadata:
       {{- with .Values.backend.clickhouseMigrations.annotations }}

diff --git a/charts/langsmith/templates/backend/postgres-migrations.yaml b/charts/langsmith/templates/backend/postgres-migrations.yaml
@@ -20,7 +20,7 @@ metadata:
     {{- end }}
 spec:
   backoffLimit: 3
-  ttlSecondsAfterFinished: 600
+  ttlSecondsAfterFinished: {{ .Values.backend.migrations.ttlSecondsAfterFinished }}
   template:
     metadata:
       {{- with .Values.backend.migrations.annotations }}

diff --git a/charts/langsmith/values.yaml b/charts/langsmith/values.yaml
@@ -289,6 +289,8 @@ backend:
     affinity: {}
     volumes: []
     volumeMounts: []
+    # You can set this to null to disable the TTL for the migrations job. Useful for some deployment setups.
+    ttlSecondsAfterFinished: 600
   authBootstrap:
     labels: {}
     annotations: {}
@@ -313,6 +315,8 @@ backend:
     affinity: {}
     volumes: []
     volumeMounts: []
+    # You can set this to null to disable the TTL for the migrations job. Useful for some deployment setups.
+    ttlSecondsAfterFinished: 600
   clickhouseMigrations:
     enabled: true
     labels: {}
@@ -338,6 +342,8 @@ backend:
     affinity: {}
     volumes: []
     volumeMounts: []
+    # You can set this to null to disable the TTL for the migrations job. Useful for some deployment setups.
+    ttlSecondsAfterFinished: 600
   autoscaling:
     enabled: false
     # If enabled, use the following values to configure the HPA. You can also use your own HPA configuration by not creating an HPA.