From 628e0b7cec07b8bdb5f6e8d4c3c0c6c4e8f58b50 Mon Sep 17 00:00:00 2001 From: Matt Butcher Date: Fri, 24 Jun 2016 12:52:30 -0600 Subject: [PATCH] fix(docs): fix nginx example chart name The name was not correctly truncating, which occasionally resulted in naming errors, since Kubernetes restricts names to 24 characters. --- docs/examples/nginx/templates/_helpers.tpl | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/examples/nginx/templates/_helpers.tpl b/docs/examples/nginx/templates/_helpers.tpl index 1af41a2e3..200aee93a 100644 --- 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}}