Commit 141a401c authored by Steve Wilkerson's avatar Steve Wilkerson

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
parent 72a93efb
...@@ -2,15 +2,15 @@ ...@@ -2,15 +2,15 @@
{{/* {{/*
Expand the name of the chart. 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. 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). (by the DNS naming spec).
*/}} */}}
{{define "fullname"}} {{define "fullname"}}
{{- $name := default "nginx" .Values.nameOverride -}} {{- $name := default "nginx" .Values.nameOverride -}}
{{printf "%s-%s" .Release.Name $name | trunc 24 | trimSuffix "-" -}} {{printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
{{end}} {{end}}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment