feat: add support for setting cluster domain (#476)

* feat: add support for setting cluster domain

* feat: add support for setting cluster domain

Author: langchain-infra

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.11.24
+version: 0.11.25
 appVersion: "0.11.69"

charts/langsmith/README.md

@@ -1,6 +1,6 @@
 # langsmith
 
-![Version: 0.11.24](https://img.shields.io/badge/Version-0.11.24-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.11.69](https://img.shields.io/badge/AppVersion-0.11.69-informational?style=flat-square)
+![Version: 0.11.25](https://img.shields.io/badge/Version-0.11.25-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.11.69](https://img.shields.io/badge/AppVersion-0.11.69-informational?style=flat-square)
 
 Helm chart to deploy the langsmith application and all services it depends on.
 
@@ -12,6 +12,7 @@ For information on how to use this chart, up-to-date release notes, and other gu
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
+| clusterDomain | string | `"cluster.local"` | Kubernetes cluster domain. Only change if not using 'cluster.local' |
 | commonAnnotations | object | `{}` | Annotations that will be applied to all resources created by the chart |
 | commonEnv | list | `[]` | Common environment variables that will be applied to all deployments/statefulsets except for the playground/aceBackend services (which are sandboxed). Be careful not to override values already specified by the chart. |
 | commonLabels | object | `{}` | Labels that will be applied to all resources created by the chart |

charts/langsmith/templates/clickhouse/secrets.yaml

@@ -20,7 +20,7 @@ data:
   {{- else }}
   clickhouse_user: {{ .Values.clickhouse.external.user | b64enc }}
   clickhouse_password: {{ .Values.clickhouse.external.password | b64enc }}
-  clickhouse_host: {{ printf "%s-%s.%s.svc.cluster.local" (include "langsmith.fullname" .) .Values.clickhouse.name .Release.Namespace  | b64enc }}
+  clickhouse_host: {{ printf "%s-%s.%s.svc.%s" (include "langsmith.fullname" .) .Values.clickhouse.name .Release.Namespace .Values.clusterDomain  | b64enc }}
   clickhouse_port: {{ toString .Values.clickhouse.containerHttpPort | b64enc }}
   clickhouse_native_port: {{ toString .Values.clickhouse.containerNativePort | b64enc }}
   clickhouse_db: {{ .Values.clickhouse.external.database | b64enc }}

charts/langsmith/templates/config-map.yaml

@@ -32,11 +32,11 @@ data:
   AUTH_TYPE: "none"
   {{- end }}
   LANGCHAIN_ENV: "local_kubernetes"
-  GO_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{ .Values.platformBackend.name }}.{{ .Values.namespace | default .Release.Namespace }}.svc.cluster.local:{{ .Values.platformBackend.service.port }}"
-  GO_ACE_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.aceBackend.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.cluster.local:{{ .Values.aceBackend.service.port }}"
-  PLAYGROUND_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.playground.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.cluster.local:{{ .Values.playground.service.port }}"
-  SMITH_BACKEND_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.backend.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.cluster.local:{{ .Values.backend.service.port }}"
-  HOST_BACKEND_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.hostBackend.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.cluster.local:{{ .Values.hostBackend.service.port }}"
+  GO_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{ .Values.platformBackend.name }}.{{ .Values.namespace | default .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.platformBackend.service.port }}"
+  GO_ACE_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.aceBackend.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.aceBackend.service.port }}"
+  PLAYGROUND_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.playground.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.playground.service.port }}"
+  SMITH_BACKEND_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.backend.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.backend.service.port }}"
+  HOST_BACKEND_ENDPOINT: "http://{{ include "langsmith.fullname" . }}-{{.Values.hostBackend.name}}.{{ .Values.namespace | default .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.hostBackend.service.port }}"
   FF_RUN_STATS_GROUP_BY_ENABLED_ALL: "true"
   SEPARATE_QUEUES_WITH_SINGLE_WORKER: '["host"]'
   {{- if .Values.config.observability.tracing.enabled }}