Skip to content

Commit

Permalink
Merge pull request #3 from koslib/features/remove-name-suffix
Browse files Browse the repository at this point in the history
Features/remove name suffix
  • Loading branch information
koslib authored Oct 16, 2022
2 parents affe11c + e802544 commit 2138c09
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ apiVersion: v2
name: servicetpl
description: A Helm chart for services running on Kubernetes

version: 0.12.2
version: 0.13.0
4 changes: 2 additions & 2 deletions templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this
If release name contains chart name it will be used as a full name.
*/}}
{{- define "service_template.fullname" -}}
{{- if .Values.fullnameOverride }}
{{- .Values.fullnameOverride | trunc 63 | trimSuffix "-" }}
{{- if .Values.serviceName }}
{{- .Values.serviceName | trunc 63 | trimSuffix "-" }}
{{- else }}
{{- $name := default .Chart.Name .Values.nameOverride }}
{{- if contains $name .Release.Name }}
Expand Down
1 change: 0 additions & 1 deletion values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ image:

imagePullSecrets: []
nameOverride: ""
fullnameOverride: ""

rollingUpdate:
maxSurge: 1
Expand Down

0 comments on commit 2138c09

Please sign in to comment.