Skip to content

Commit

Permalink
When providing the UID or GID as a number the chart fails to deploy, …
Browse files Browse the repository at this point in the history
…adding quotations to where extraEnv lands the extra environment variables fixes the issue

Updating method to quote the env values per PR conversation, updating chart version with a patch bump
  • Loading branch information
Joel Chapman authored and Joel Chapman committed Nov 14, 2023
1 parent e557be3 commit 7b961bb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/plex-media-server/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.6
version: 0.1.7

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
2 changes: 1 addition & 1 deletion charts/plex-media-server/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ spec:
{{- end }}
{{- range $key, $value := .Values.extraEnv }}
- name: {{ $key }}
value: {{ $value }}
value: {{ $value | quote }}
{{- end }}
{{- with .Values.pms.resources }}
resources:
Expand Down

0 comments on commit 7b961bb

Please sign in to comment.