Merge pull request #883 from technosophos/fix/879-fix-name-truncate

fix(docs): fix nginx example chart name
8 years ago · 100dd314f2
parent 59806b40a8 628e0b7cec
commit 100dd314f2
1 changed files with 4 additions and 1 deletions
--- a/docs/examples/nginx/templates/_helpers.tpl
+++ b/docs/examples/nginx/templates/_helpers.tpl
@ -10,4 +10,7 @@ Create a default fully qualified app name.
 We truncate at 24 chars because some Kubernetes name fields are limited to this
 (by the DNS naming spec).
 */}}
-{{define "fullname"}}{{.Release.Name}}-{{default "nginx" .Values.nameOverride | trunc 24 }}{{end}}
+{{define "fullname"}}
+{{- $name := default "nginx" .Values.nameOverride -}}
+{{printf "%s-%s" .Release.Name $name | trunc 24 -}}
+{{end}}