Skip to content

Commit

Permalink
fix: add termination grace period seconds to chart (#191)
Browse files Browse the repository at this point in the history
  • Loading branch information
langchain-infra authored Nov 28, 2024
1 parent e6778b5 commit 96d7860
Show file tree
Hide file tree
Showing 9 changed files with 33 additions and 15 deletions.
4 changes: 2 additions & 2 deletions charts/langsmith/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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.15
appVersion: "0.844"
version: 0.8.16
appVersion: "0.8.47"
22 changes: 14 additions & 8 deletions charts/langsmith/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# langsmith

![Version: 0.8.15](https://img.shields.io/badge/Version-0.8.15-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.844](https://img.shields.io/badge/AppVersion-0.844-informational?style=flat-square)
![Version: 0.8.16](https://img.shields.io/badge/Version-0.8.16-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.8.47](https://img.shields.io/badge/AppVersion-0.8.47-informational?style=flat-square)

Helm chart to deploy the langsmith application and all services it depends on.

Expand Down Expand Up @@ -60,6 +60,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
| aceBackend.deployment.startupProbe.httpGet.port | int | `1987` | |
| aceBackend.deployment.startupProbe.periodSeconds | int | `10` | |
| aceBackend.deployment.startupProbe.timeoutSeconds | int | `1` | |
| aceBackend.deployment.terminationGracePeriodSeconds | int | `30` | |
| aceBackend.deployment.tolerations | list | `[]` | |
| aceBackend.deployment.topologySpreadConstraints | list | `[]` | |
| aceBackend.deployment.volumeMounts | list | `[]` | |
Expand Down Expand Up @@ -90,7 +91,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.hybrid | bool | `false` | Set to true if using managed ClickHouse |
| clickhouse.external.hybrid | bool | `false` | Must be set to true if using managed ClickHouse |
| clickhouse.external.nativePort | string | `"9000"` | |
| clickhouse.external.password | string | `"password"` | |
| clickhouse.external.port | string | `"8123"` | |
Expand Down Expand Up @@ -151,23 +152,23 @@ For information on how to use this chart, up-to-date release notes, and other gu
| fullnameOverride | string | `""` | String to fully override `"langsmith.fullname"` |
| images.aceBackendImage.pullPolicy | string | `"IfNotPresent"` | |
| images.aceBackendImage.repository | string | `"docker.io/langchain/langsmith-ace-backend"` | |
| images.aceBackendImage.tag | string | `"0.8.44"` | |
| images.aceBackendImage.tag | string | `"0.8.47"` | |
| images.backendImage.pullPolicy | string | `"IfNotPresent"` | |
| images.backendImage.repository | string | `"docker.io/langchain/langsmith-backend"` | |
| images.backendImage.tag | string | `"0.8.44"` | |
| images.backendImage.tag | string | `"0.8.47"` | |
| images.clickhouseImage.pullPolicy | string | `"Always"` | |
| images.clickhouseImage.repository | string | `"docker.io/clickhouse/clickhouse-server"` | |
| images.clickhouseImage.tag | string | `"24.5"` | |
| images.frontendImage.pullPolicy | string | `"IfNotPresent"` | |
| images.frontendImage.repository | string | `"docker.io/langchain/langsmith-frontend"` | |
| images.frontendImage.tag | string | `"0.8.44"` | |
| images.frontendImage.tag | string | `"0.8.47"` | |
| images.imagePullSecrets | list | `[]` | Secrets with credentials to pull images from a private registry. Specified as name: value. |
| images.platformBackendImage.pullPolicy | string | `"IfNotPresent"` | |
| images.platformBackendImage.repository | string | `"docker.io/langchain/langsmith-go-backend"` | |
| images.platformBackendImage.tag | string | `"0.8.44"` | |
| images.platformBackendImage.tag | string | `"0.8.47"` | |
| images.playgroundImage.pullPolicy | string | `"IfNotPresent"` | |
| images.playgroundImage.repository | string | `"docker.io/langchain/langsmith-playground"` | |
| images.playgroundImage.tag | string | `"0.8.44"` | |
| images.playgroundImage.tag | string | `"0.8.47"` | |
| images.postgresImage.pullPolicy | string | `"IfNotPresent"` | |
| images.postgresImage.repository | string | `"docker.io/postgres"` | |
| images.postgresImage.tag | string | `"14.7"` | |
Expand Down Expand Up @@ -303,6 +304,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
| backend.deployment.startupProbe.httpGet.port | int | `1984` | |
| backend.deployment.startupProbe.periodSeconds | int | `10` | |
| backend.deployment.startupProbe.timeoutSeconds | int | `1` | |
| backend.deployment.terminationGracePeriodSeconds | int | `30` | |
| backend.deployment.tolerations | list | `[]` | |
| backend.deployment.topologySpreadConstraints | list | `[]` | |
| backend.deployment.volumeMounts | list | `[]` | |
Expand Down Expand Up @@ -352,7 +354,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.hybrid | bool | `false` | Set to true if using managed ClickHouse |
| clickhouse.external.hybrid | bool | `false` | Must be set to true if using managed ClickHouse |
| clickhouse.external.nativePort | string | `"9000"` | |
| clickhouse.external.password | string | `"password"` | |
| clickhouse.external.port | string | `"8123"` | |
Expand Down Expand Up @@ -450,6 +452,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
| frontend.deployment.startupProbe.httpGet.port | int | `8080` | |
| frontend.deployment.startupProbe.periodSeconds | int | `10` | |
| frontend.deployment.startupProbe.timeoutSeconds | int | `1` | |
| frontend.deployment.terminationGracePeriodSeconds | int | `30` | |
| frontend.deployment.tolerations | list | `[]` | |
| frontend.deployment.topologySpreadConstraints | list | `[]` | |
| frontend.deployment.volumeMounts | list | `[]` | |
Expand Down Expand Up @@ -514,6 +517,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
| platformBackend.deployment.startupProbe.httpGet.port | int | `1986` | |
| platformBackend.deployment.startupProbe.periodSeconds | int | `10` | |
| platformBackend.deployment.startupProbe.timeoutSeconds | int | `1` | |
| platformBackend.deployment.terminationGracePeriodSeconds | int | `30` | |
| platformBackend.deployment.tolerations | list | `[]` | |
| platformBackend.deployment.topologySpreadConstraints | list | `[]` | |
| platformBackend.deployment.volumeMounts | list | `[]` | |
Expand Down Expand Up @@ -585,6 +589,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
| playground.deployment.startupProbe.httpGet.port | int | `1988` | |
| playground.deployment.startupProbe.periodSeconds | int | `10` | |
| playground.deployment.startupProbe.timeoutSeconds | int | `1` | |
| playground.deployment.terminationGracePeriodSeconds | int | `30` | |
| playground.deployment.tolerations | list | `[]` | |
| playground.deployment.topologySpreadConstraints | list | `[]` | |
| playground.deployment.volumeMounts | list | `[]` | |
Expand Down Expand Up @@ -711,6 +716,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
| queue.deployment.startupProbe.failureThreshold | int | `6` | |
| queue.deployment.startupProbe.periodSeconds | int | `60` | |
| queue.deployment.startupProbe.timeoutSeconds | int | `30` | |
| queue.deployment.terminationGracePeriodSeconds | int | `30` | |
| queue.deployment.tolerations | list | `[]` | |
| queue.deployment.topologySpreadConstraints | list | `[]` | |
| queue.deployment.volumeMounts | list | `[]` | |
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/ace-backend/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ spec:
{{- include "langsmith.labels" . | nindent 8 }}
app.kubernetes.io/component: {{ include "langsmith.fullname" . }}-{{ .Values.aceBackend.name }}
spec:
terminationGracePeriodSeconds: {{ .Values.aceBackend.deployment.terminationGracePeriodSeconds }}
{{- with .Values.images.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/backend/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ spec:
{{- include "langsmith.labels" . | nindent 8 }}
app.kubernetes.io/component: {{ include "langsmith.fullname" . }}-{{ .Values.backend.name }}
spec:
terminationGracePeriodSeconds: {{ .Values.backend.deployment.terminationGracePeriodSeconds }}
{{- with .Values.images.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/frontend/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ spec:
{{- include "langsmith.labels" . | nindent 8 }}
app.kubernetes.io/component: {{ include "langsmith.fullname" . }}-{{ .Values.frontend.name }}
spec:
terminationGracePeriodSeconds: {{ .Values.frontend.deployment.terminationGraceSeconds }}
{{- with .Values.images.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ spec:
{{- include "langsmith.labels" . | nindent 8 }}
app.kubernetes.io/component: {{ include "langsmith.fullname" . }}-{{ .Values.platformBackend.name }}
spec:
terminationGracePeriodSeconds: {{ .Values.platformBackend.deployment.terminationGracePeriodSeconds }}
{{- with .Values.images.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/playground/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ spec:
{{- include "langsmith.labels" . | nindent 8 }}
app.kubernetes.io/component: {{ include "langsmith.fullname" . }}-{{ .Values.playground.name }}
spec:
terminationGracePeriodSeconds: {{ .Values.playground.deployment.terminationGracePeriodSeconds }}
{{- with .Values.images.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/queue/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ spec:
{{- include "langsmith.labels" . | nindent 8 }}
app.kubernetes.io/component: {{ include "langsmith.fullname" . }}-{{ .Values.queue.name }}
spec:
terminationGracePeriodSeconds: {{ .Values.queue.deployment.terminationGracePeriodSeconds }}
{{- with .Values.images.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
Expand Down
16 changes: 11 additions & 5 deletions charts/langsmith/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,23 +17,23 @@ images:
aceBackendImage:
repository: "docker.io/langchain/langsmith-ace-backend"
pullPolicy: IfNotPresent
tag: "0.8.44"
tag: "0.8.47"
backendImage:
repository: "docker.io/langchain/langsmith-backend"
pullPolicy: IfNotPresent
tag: "0.8.44"
tag: "0.8.47"
frontendImage:
repository: "docker.io/langchain/langsmith-frontend"
pullPolicy: IfNotPresent
tag: "0.8.44"
tag: "0.8.47"
platformBackendImage:
repository: "docker.io/langchain/langsmith-go-backend"
pullPolicy: IfNotPresent
tag: "0.8.44"
tag: "0.8.47"
playgroundImage:
repository: "docker.io/langchain/langsmith-playground"
pullPolicy: IfNotPresent
tag: "0.8.44"
tag: "0.8.47"
postgresImage:
repository: "docker.io/postgres"
pullPolicy: IfNotPresent
Expand Down Expand Up @@ -194,6 +194,7 @@ aceBackend:
affinity: {}
volumes: []
volumeMounts: []
terminationGracePeriodSeconds: 30
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.
Expand Down Expand Up @@ -279,6 +280,7 @@ backend:
affinity: {}
volumes: []
volumeMounts: []
terminationGracePeriodSeconds: 30
migrations:
enabled: true
# Helpful when running using helm template to avoid the job name conflict
Expand Down Expand Up @@ -535,6 +537,7 @@ frontend:
affinity: {}
volumes: []
volumeMounts: []
terminationGracePeriodSeconds: 30
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.
Expand Down Expand Up @@ -608,6 +611,7 @@ platformBackend:
affinity: {}
volumes: []
volumeMounts: []
terminationGracePeriodSeconds: 30
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.
Expand Down Expand Up @@ -691,6 +695,7 @@ playground:
affinity: {}
volumes: []
volumeMounts: []
terminationGracePeriodSeconds: 30
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.
Expand Down Expand Up @@ -852,6 +857,7 @@ queue:
affinity: {}
volumes: []
volumeMounts: []
terminationGracePeriodSeconds: 30
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.
Expand Down

0 comments on commit 96d7860

Please sign in to comment.