From 36afd515daea0cead0262e6c935e2c1b2b0161f4 Mon Sep 17 00:00:00 2001 From: infra Date: Wed, 27 Nov 2024 22:20:24 -0500 Subject: [PATCH] fix: add termination grace period seconds to chart --- charts/langsmith/Chart.yaml | 4 ++-- charts/langsmith/README.md | 22 ++++++++++++------- .../templates/ace-backend/deployment.yaml | 1 + .../templates/backend/deployment.yaml | 1 + .../templates/frontend/deployment.yaml | 1 + .../platform-backend/deployment.yaml | 1 + .../templates/playground/deployment.yaml | 1 + .../langsmith/templates/queue/deployment.yaml | 1 + charts/langsmith/values.yaml | 16 +++++++++----- 9 files changed, 33 insertions(+), 15 deletions(-) diff --git a/charts/langsmith/Chart.yaml b/charts/langsmith/Chart.yaml index 0fa1f5a..d44c254 100644 --- a/charts/langsmith/Chart.yaml +++ b/charts/langsmith/Chart.yaml @@ -5,5 +5,5 @@ maintainers: email: ankush@langchain.dev 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" diff --git a/charts/langsmith/README.md b/charts/langsmith/README.md index 98e4f79..21cf3e6 100644 --- a/charts/langsmith/README.md +++ b/charts/langsmith/README.md @@ -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. @@ -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 | `[]` | | @@ -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"` | | @@ -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"` | | @@ -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 | `[]` | | @@ -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"` | | @@ -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 | `[]` | | @@ -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 | `[]` | | @@ -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 | `[]` | | @@ -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 | `[]` | | diff --git a/charts/langsmith/templates/ace-backend/deployment.yaml b/charts/langsmith/templates/ace-backend/deployment.yaml index 52c7d3b..b54fe0e 100644 --- a/charts/langsmith/templates/ace-backend/deployment.yaml +++ b/charts/langsmith/templates/ace-backend/deployment.yaml @@ -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 }} diff --git a/charts/langsmith/templates/backend/deployment.yaml b/charts/langsmith/templates/backend/deployment.yaml index b42f9b3..9ff9c72 100644 --- a/charts/langsmith/templates/backend/deployment.yaml +++ b/charts/langsmith/templates/backend/deployment.yaml @@ -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 }} diff --git a/charts/langsmith/templates/frontend/deployment.yaml b/charts/langsmith/templates/frontend/deployment.yaml index 3ccefd5..12fa10e 100644 --- a/charts/langsmith/templates/frontend/deployment.yaml +++ b/charts/langsmith/templates/frontend/deployment.yaml @@ -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 }} diff --git a/charts/langsmith/templates/platform-backend/deployment.yaml b/charts/langsmith/templates/platform-backend/deployment.yaml index c934ebf..310d5be 100644 --- a/charts/langsmith/templates/platform-backend/deployment.yaml +++ b/charts/langsmith/templates/platform-backend/deployment.yaml @@ -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 }} diff --git a/charts/langsmith/templates/playground/deployment.yaml b/charts/langsmith/templates/playground/deployment.yaml index 54fe3f6..75b044a 100644 --- a/charts/langsmith/templates/playground/deployment.yaml +++ b/charts/langsmith/templates/playground/deployment.yaml @@ -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 }} diff --git a/charts/langsmith/templates/queue/deployment.yaml b/charts/langsmith/templates/queue/deployment.yaml index f2002b2..37c0626 100644 --- a/charts/langsmith/templates/queue/deployment.yaml +++ b/charts/langsmith/templates/queue/deployment.yaml @@ -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 }} diff --git a/charts/langsmith/values.yaml b/charts/langsmith/values.yaml index 86d8a2d..92ed8ef 100644 --- a/charts/langsmith/values.yaml +++ b/charts/langsmith/values.yaml @@ -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 @@ -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. @@ -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 @@ -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. @@ -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. @@ -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. @@ -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.