add labels templating

vmware · Jan 23, 2023 · e5f223b · e5f223b
1 parent 4a93dc9
commit e5f223b
Show file tree

Hide file tree

Showing 9 changed files with 47 additions and 53 deletions.
diff --git a/charts/log-router/templates/_helpers.tpl b/charts/log-router/templates/_helpers.tpl
@@ -19,6 +19,33 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this
 {{- end -}}
 {{- end -}}
 
+{{/*
+Create chart name and version as used by the chart label.
+*/}}
+{{- define "fluentd-router.chart" -}}
+{{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" }}
+{{- end }}
+
+{{/*
+Common labels
+*/}}
+{{- define "fluentd-router.labels" -}}
+helm.sh/chart: {{ include "fluentd-router.chart" . }}
+{{ include "fluentd-router.selectorLabels" . }}
+{{- if .Chart.AppVersion }}
+app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
+{{- end }}
+app.kubernetes.io/managed-by: {{ .Release.Service }}
+{{- end }}
+
+{{/*
+Selector labels
+*/}}
+{{- define "fluentd-router.selectorLabels" -}}
+app.kubernetes.io/name: {{ include "fluentd-router.name" . }}
+app.kubernetes.io/instance: {{ .Release.Name }}
+{{- end }}
+
 {{/*
 Set apiVersion based on .Capabilities.APIVersions
 */}}

diff --git a/charts/log-router/templates/clusterrole.yaml b/charts/log-router/templates/clusterrole.yaml
@@ -8,10 +8,7 @@ apiVersion: {{ template "rbacAPIVersion" . }}
 kind: ClusterRole
 metadata:
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version }}
-    heritage: {{ .Release.Service }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
     {{- if .Values.extraLabels }}
 {{ toYaml .Values.extraLabels | indent 4 }}
     {{- end }}

diff --git a/charts/log-router/templates/daemonset.yaml b/charts/log-router/templates/daemonset.yaml
@@ -8,20 +8,15 @@ kind: DaemonSet
 metadata:
   name: {{ template "fluentd-router.fullname" . }}
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
-    release: {{ .Release.Name }}
-    heritage: {{ .Release.Service }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
 spec:
   selector:
     matchLabels:
-      app: {{ template "fluentd-router.name" . }}
-      release: {{ .Release.Name }}
+      {{- include "fluentd-router.selectorLabels" . | nindent 6 }}
   template:
     metadata:
       labels:
-        app: {{ template "fluentd-router.name" . }}
-        release: {{ .Release.Name }}
+        {{- include "fluentd-router.selectorLabels" . | nindent 8 }}
       annotations:
         checksum/fluentd-extraenv: {{ toYaml .Values.fluentd.extraEnv | sha256sum }}
         checksum/reloader-extraenv: {{ toYaml .Values.reloader.extraEnv | sha256sum }}

diff --git a/charts/log-router/templates/rolebinding.yaml b/charts/log-router/templates/rolebinding.yaml
@@ -8,10 +8,7 @@ apiVersion: {{ template "rbacAPIVersion" . }}
 kind: ClusterRoleBinding
 metadata:
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version }}
-    heritage: {{ .Release.Service }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
     {{- if .Values.extraLabels }}
 {{ toYaml .Values.extraLabels | indent 4 }}
     {{- end }}

diff --git a/charts/log-router/templates/sa.yaml b/charts/log-router/templates/sa.yaml
@@ -7,10 +7,7 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version }}
-    heritage: {{ .Release.Service }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
   name: {{ template "fluentd-router.fullname" . }}
   {{- if .Values.rbac.serviceAccount.annotations }}
   annotations: {{ toYaml .Values.rbac.serviceAccount.annotations | nindent 4 }}
@@ -19,4 +16,4 @@ metadata:
 imagePullSecrets:
   - name: {{ .Values.imagePullSecret}}
 {{end}}
-{{- end }}
+{{- end }}
diff --git a/charts/log-router/templates/secret.yaml b/charts/log-router/templates/secret.yaml
@@ -8,10 +8,7 @@ kind: Secret
 metadata:
   name: {{ template "fluentd-router.fullname" . }}
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
-    release: {{ .Release.Name }}
-    heritage: {{ .Release.Service }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
   annotations:
     checksum/fluentd-extraenv: {{ toYaml .Values.fluentd.extraEnv | sha256sum }}
     checksum/reloader-extraenv: {{ toYaml .Values.reloader.extraEnv | sha256sum }}

diff --git a/charts/log-router/templates/service.yaml b/charts/log-router/templates/service.yaml
@@ -7,19 +7,15 @@ apiVersion: v1
 kind: Service
 metadata:
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version }}
-    heritage: {{ .Release.Service }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
     metrics: fluentd
   annotations:
     prometheus.io/scrape: "true"
     prometheus.io/port: "24231"
   name: {{ template "fluentd-router.fullname" . }}
 spec:
   selector:
-    app: {{ template "fluentd-router.name" . }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.selectorLabels" . | nindent 4 }}
   ports:
     - port: 24231
       name: prometheus
@@ -31,21 +27,17 @@ apiVersion: v1
 kind: Service
 metadata:
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version }}
-    heritage: {{ .Release.Service }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
     metrics: reloader
   annotations:
     prometheus.io/scrape: "true"
     prometheus.io/port: {{ .Values.metricsPort | quote }}
   name: {{ template "fluentd-router.fullname" . }}-reloader
 spec:
   selector:
-    app: {{ template "fluentd-router.name" . }}
-    release: {{ .Release.Name }}
+    {{- include "fluentd-router.selectorLabels" . | nindent 4 }}
   ports:
     - port: {{ .Values.metricsPort }}
       name: metrics
       targetPort: metrics
-{{- end }}
+{{- end }}
diff --git a/charts/log-router/templates/servicemonitor.yaml b/charts/log-router/templates/servicemonitor.yaml
@@ -4,16 +4,12 @@ kind: ServiceMonitor
 metadata:
   name: {{ template "fluentd-router.fullname" . }}
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
-    release: {{ .Release.Name }}
-    heritage: {{ .Release.Service }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
 spec:
-  jobLabel: app
+  jobLabel: {{ .Values.serviceMonitor.jobLabel | quote }}
   selector:
     matchLabels:
-      app: {{ template "fluentd-router.name" . }}
-      release: {{ .Release.Name }}
+      {{- include "fluentd-router.selectorLabels" . | nindent 6 }}
       metrics: fluentd
   endpoints:
   - port: prometheus
@@ -28,16 +24,12 @@ kind: ServiceMonitor
 metadata:
   name: {{ template "fluentd-router.fullname" . }}-reloader
   labels:
-    app: {{ template "fluentd-router.name" . }}
-    chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
-    release: {{ .Release.Name }}
-    heritage: {{ .Release.Service }}
+    {{- include "fluentd-router.labels" . | nindent 4 }}
 spec:
-  jobLabel: app
+  jobLabel: {{ .Values.serviceMonitor.jobLabel | quote }}
   selector:
     matchLabels:
-      app: {{ template "fluentd-router.name" . }}
-      release: {{ .Release.Name }}
+      {{- include "fluentd-router.selectorLabels" . | nindent 6 }}
       metrics: reloader
   endpoints:
   - port: metrics

diff --git a/charts/log-router/values.yaml b/charts/log-router/values.yaml
@@ -120,7 +120,7 @@ serviceMonitor:
     enabled: false
 
     # Job label for scrape target
-    jobLabel: app
+  jobLabel: app.kubernetes.io/name
 
     # Scrape interval. If not set, the Prometheus default scrape interval is used.
     interval: ""