Skip to content

feat: add envFrom value to Helm chart #9587

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Sep 19, 2023
Merged

feat: add envFrom value to Helm chart #9587

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
797c08d
feat: add envFrom value to Helm chart
ericpaulsen Sep 7, 2023
65b78ef
add envUseClusterAccessURL
ericpaulsen Sep 13, 2023
cb32ab1
Merge branch 'main' into helm-envFrom-value
ericpaulsen Sep 13, 2023
1ab8d78
add tests
deansheather Sep 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion helm/coder/templates/_coder.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,12 @@ args:
{{- end }}
- server
{{- end }}
{{- if .Values.coder.envFrom }}
envFrom:
{{- with .Values.coder.envFrom }}
{{ toYaml . }}
{{- end }}
{{- end }}
env:
- name: CODER_HTTP_ADDRESS
value: "0.0.0.0:8080"
Expand All @@ -50,7 +56,7 @@ env:
{{- $hasAccessURL = true }}
{{- end }}
{{- end }}
{{- if not $hasAccessURL }}
{{- if and (not $hasAccessURL) .Values.coder.envUseClusterAccessURL }}
- name: CODER_ACCESS_URL
value: {{ include "coder.defaultAccessURL" . | quote }}
{{- end }}
Expand Down
17 changes: 14 additions & 3 deletions helm/coder/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,22 @@ coder:
# - CODER_DERP_SERVER_RELAY_URL
#
# We will additionally set CODER_ACCESS_URL if unset to the cluster service
# URL.
# URL, unless coder.envUseClusterAccessURL is set to false.
env: []
# - name: "CODER_ACCESS_URL"
# value: "https://coder.example.com"

# coder.envFrom -- Secrets to use for Coder's environment variables. If you want
# one environment variable read from a secret, then use coder.env valueFrom. See
# the K8s docs here: https://kubernetes.io/docs/tasks/inject-data-application/distribute-credentials-secure/#define-container-environment-variables-using-secret-data
# If setting CODER_ACCESS_URL in coder.envFrom, then you must set coder.envUseClusterAccessURL
# to false.
envFrom: []

# coder.envUseClusterAccessURL -- Set this to false if defining CODER_ACCESS_URL
# in coder.envFrom
envUseClusterAccessURL: true

# coder.image -- The image to use for Coder.
image:
# coder.image.repo -- The repository of the image.
Expand All @@ -29,13 +40,13 @@ coder:
# if not set. If you're using the chart directly from git, the default
# app version will not work and you'll need to set this value. The helm
# chart helpfully fails quickly in this case.
tag: ""
tag: "v2.1.5"
# coder.image.pullPolicy -- The pull policy to use for the image. See:
# https://kubernetes.io/docs/concepts/containers/images/#image-pull-policy
pullPolicy: IfNotPresent
# coder.image.pullSecrets -- The secrets used for pulling the Coder image from
# a private registry.
pullSecrets: []
pullSecrets:
# - name: "pull-secret"

# coder.initContainers -- Init containers for the deployment. See:
Expand Down