Skip to content

Commit

Permalink
Merge pull request #44 from directus-labs/attach-secrets
Browse files Browse the repository at this point in the history
Attach Kubernetes Secrets to the Directus pod
  • Loading branch information
mikesindieiev authored Oct 30, 2024
2 parents dbc7f19 + faafd5d commit 659a951
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 3 deletions.
4 changes: 2 additions & 2 deletions charts/directus/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ maintainers:
email: [email protected]
url: https://github.com/directus-labs/helm-chart

version: 1.0.3
appVersion: "11.1.1"
version: 1.0.4
appVersion: "11.1.2"

dependencies:
- name: mariadb
Expand Down
4 changes: 4 additions & 0 deletions charts/directus/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@ spec:
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: {{ .Values.image.pullPolicy }}
envFrom:
{{- range $val := .Values.attachExistingSecrets }}
- secretRef:
name: {{ $val }}
{{- end }}
- configMapRef:
name: {{ include "directus.fullname" . }}-configmap
env:
Expand Down
5 changes: 4 additions & 1 deletion charts/directus/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ image:
repository: directus/directus
pullPolicy: IfNotPresent
# Overrides the image tag whose default is the chart appVersion.
tag: "11.1.1"
tag: "11.1.2"

imagePullSecrets: []
nameOverride: ""
Expand Down Expand Up @@ -72,6 +72,8 @@ ingress:

extraEnvVars: []

attachExistingSecrets: []
# - existingSecretInKubernetes

resources: {}
# We usually recommend not to specify default resources and to leave this as a conscious
Expand Down Expand Up @@ -119,6 +121,7 @@ extraVolumes: []

extraVolumeMounts: []


mariadb:
# The switch to switch off the installation of the mariadb, the rest of the settings are being used during the installation
enableInstallation: true
Expand Down

0 comments on commit 659a951

Please sign in to comment.