Skip to content

H
helm3
Commit 48553d0a authored by Matt Butcher's avatar Matt Butcher Committed by GitHub
Browse files
Options

Merge pull request #1740 from wilkers-steve/docs/change_trunc_length 

docs(helm): change `trunc 24` in base charts
parents b51ad7a0 0d9c6bee
Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 3 deletions
docs/examples/nginx/templates/_helpers.tpl
View file @ 48553d0a
...@@ -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