From 141a401c480aa4298ce1dcd4111cf9ccedf6b070 Mon Sep 17 00:00:00 2001 From: Steve Wilkerson Date: Tue, 20 Dec 2016 16:35:46 -0600 Subject: [PATCH] docs(helm): change `trunc 24` in base charts The upper limit for a chart name is 63 characters now instead of 14 or 24 in older versions of Kubernetes. This replaces `trunc 24` in the example chart provided to `trunc 63` to reflect the new length available. Closes #1637 --- docs/examples/nginx/templates/_helpers.tpl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/examples/nginx/templates/_helpers.tpl b/docs/examples/nginx/templates/_helpers.tpl index 11d18431d..24f76db73 100644 --- a/docs/examples/nginx/templates/_helpers.tpl +++ b/docs/examples/nginx/templates/_helpers.tpl @@ -2,15 +2,15 @@ {{/* Expand the name of the chart. */}} -{{define "name"}}{{default "nginx" .Values.nameOverride | trunc 24 | trimSuffix "-" }}{{end}} +{{define "name"}}{{default "nginx" .Values.nameOverride | trunc 63 | trimSuffix "-" }}{{end}} {{/* Create a default fully qualified app name. -We truncate at 24 chars because some Kubernetes name fields are limited to this +We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec). */}} {{define "fullname"}} {{- $name := default "nginx" .Values.nameOverride -}} -{{printf "%s-%s" .Release.Name $name | trunc 24 | trimSuffix "-" -}} +{{printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}} {{end}}