Merge pull request #254 from nlamirault/feat/labels

feat(helm): refactoring Kubernetes labels

helm/templates/_helpers.tpl

@@ -51,6 +51,11 @@ helm.sh/chart: {{ include "kro.chart" . }}
 app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 {{- end }}
 app.kubernetes.io/managed-by: {{ .Release.Service }}
+app.kubernetes.io/component: controller
+app.kubernetes.io/part-of: kro
+{{- if .Values.additionalLabels }}
+{{ toYaml .Values.additionalLabels }}
+{{- end }}
 {{- end }}
 
 {{/*

helm/templates/cluster-role-binding.yaml

@@ -3,13 +3,12 @@ kind: ClusterRoleBinding
 metadata:
   name: {{ include "kro.fullname" . }}-role-binding
   labels:
-    app: kro
-    release: {{ .Release.Name }}
+    {{- include "kro.labels" . | nindent 4 }}
 roleRef:
   kind: ClusterRole
   apiGroup: rbac.authorization.k8s.io
   name: {{ include "kro.fullname" . }}-cluster-role
 subjects:
 - kind: ServiceAccount
   name: {{ include "kro.serviceAccountName" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ .Release.Namespace }}

helm/templates/cluster-role.yaml

@@ -2,10 +2,12 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata:
   name: {{ include "kro.fullname" . }}-cluster-role
+  labels:
+    {{- include "kro.labels" . | nindent 4 }}
 rules:
 - apiGroups:
   - "*"
   resources:
   - "*"
   verbs:
-  - '*'
+  - '*'

helm/templates/deployment.yaml

@@ -17,7 +17,7 @@ spec:
         {{- toYaml . | nindent 8 }}
       {{- end }}
       labels:
-        {{- include "kro.selectorLabels" . | nindent 8 }}
+        {{- include "kro.labels" . | nindent 8 }}
         {{- with .Values.deployment.podLabels }}
         {{- toYaml . | nindent 8 }}
         {{- end }}

helm/values.yaml

@@ -3,6 +3,10 @@ nameOverride: ""
 # Override the full name of the chart
 fullnameOverride: ""
 
+# Additional labels that should be applied to all resources
+additionalLabels: {}
+  # app: kro
+
 image:
   # The location of the container image repository
   repository: ghcr.io/kro-run/kro/controller