Skip to content

Commit

Permalink
Merge pull request #62 from langchain-ai/infra/fix-service-account-names
Browse files Browse the repository at this point in the history
Fix service account in backend
  • Loading branch information
langchain-infra authored Jan 31, 2024
2 parents dd82773 + d52a968 commit 45b31b9
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion 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.2.8
version: 0.2.9
appVersion: "0.1.0"
3 changes: 2 additions & 1 deletion charts/langsmith/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# langsmith

![Version: 0.2.7](https://img.shields.io/badge/Version-0.2.7-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.1.0](https://img.shields.io/badge/AppVersion-0.1.0-informational?style=flat-square)
![Version: 0.2.9](https://img.shields.io/badge/Version-0.2.9-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 0.1.0](https://img.shields.io/badge/AppVersion-0.1.0-informational?style=flat-square)

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

Expand Down Expand Up @@ -284,6 +284,7 @@ We typically validate deployment using the following quickstart guide:
|-----|------|---------|-------------|
| config.existingSecretName | string | `""` | |
| config.langsmithLicenseKey | string | `""` | |
| config.logLevel | string | `"info"` | |
| config.oauth.enabled | bool | `false` | |
| config.oauth.oauthClientId | string | `""` | |
| config.oauth.oauthIssuerUrl | string | `""` | |
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 @@ -39,6 +39,7 @@ spec:
{{- end }}
securityContext:
{{- toYaml .Values.backend.deployment.podSecurityContext | nindent 8 }}
serviceAccountName: {{ include "backend.serviceAccountName" . }}
containers:
- name: {{ .Values.backend.name }}
env:
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/backend/run-migrations.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ spec:
{{- end }}
securityContext:
{{- toYaml .Values.backend.migrations.podSecurityContext | nindent 8 }}
serviceAccountName: {{ include "backend.serviceAccountName" . }}
containers:
- name: {{ .Values.backend.name }}
command: ["/bin/bash", "-c", "alembic upgrade head"]
Expand Down
1 change: 1 addition & 0 deletions charts/langsmith/templates/clickhouse/stateful-set.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ spec:
{{- end }}
securityContext:
{{- toYaml .Values.clickhouse.statefulSet.podSecurityContext | nindent 8 }}
serviceAccountName: {{ include "clickhouse.serviceAccountName" . }}
containers:
- name: {{ .Values.clickhouse.name }}
image: "{{ .Values.images.clickhouseImage.repository }}:{{ .Values.images.clickhouseImage.tag | default .Chart.AppVersion }}"
Expand Down

0 comments on commit 45b31b9

Please sign in to comment.