feat(helm): clean Kubernetes labels

chart/operator/templates/_helpers.tpl

@@ -40,6 +40,10 @@ helm.sh/chart: {{ include "chart.chart" . }}
 app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 {{- end }}
 app.kubernetes.io/managed-by: {{ .Release.Service }}
+app.kubernetes.io/part-of: k8sgpt-operator
+{{- if .Values.additionalLabels}}
+{{ toYaml .Values.additionalLabels }}
+{{- end }}
 {{- end }}
 
 {{/*

chart/operator/templates/controller-manager-metrics-monitor.yaml

@@ -5,11 +5,9 @@ metadata:
   name: {{ include "chart.fullname" . }}-controller-manager-metrics-monitor
   namespace: {{ .Values.serviceMonitor.namespace | default (include "k8sgpt-operator.namespace" . ) }}
   labels:
-    app.kubernetes.io/component: metrics
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
     control-plane: controller-manager
     {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: metrics
   {{- if .Values.serviceMonitor.additionalLabels }}
     {{- toYaml .Values.serviceMonitor.additionalLabels | nindent 4 }}
   {{- end }}
@@ -29,4 +27,5 @@ spec:
   selector:
     matchLabels:
       control-plane: controller-manager
+      {{- include "chart.selectorLabels" . | nindent 6 }}
 {{ end -}}

chart/operator/templates/deployment.yaml

@@ -3,32 +3,29 @@ kind: ServiceAccount
 metadata:
   name: {{ include "chart.fullname" . }}-controller-manager
   labels:
-    app.kubernetes.io/component: rbac
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: controller-manager
 ---
 apiVersion: apps/v1
 kind: Deployment
 metadata:
   name: {{ include "chart.fullname" . }}-controller-manager
   labels:
-    app.kubernetes.io/component: manager
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
     control-plane: controller-manager
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: controller-manager
 spec:
   replicas: {{ .Values.controllerManager.replicas }}
   selector:
     matchLabels:
       control-plane: controller-manager
-    {{- include "chart.selectorLabels" . | nindent 6 }}
+      {{- include "chart.selectorLabels" . | nindent 6 }}
   template:
     metadata:
       labels:
         control-plane: controller-manager
-      {{- include "chart.selectorLabels" . | nindent 8 }}
+        {{- include "chart.labels" . | nindent 8 }}
+        app.kubernetes.io/component: controller-manager
       annotations:
         kubectl.kubernetes.io/default-container: manager
     spec:

chart/operator/templates/grafana-k8sgpt-dashboard.yaml

@@ -6,10 +6,8 @@ metadata:
     {{ .Values.grafanaDashboard.folder.annotation }}: {{ .Values.grafanaDashboard.folder.name }}
   labels:
     {{ .Values.grafanaDashboard.label.key }}: {{ .Values.grafanaDashboard.label.value | quote }}
-    app.kubernetes.io/component: configmap
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: dashboard
   name: k8sgpt-overview
   namespace: {{ .Values.grafanaDashboard.namespace | default (include "k8sgpt-operator.namespace" .) }}
 data:
@@ -21,10 +19,8 @@ apiVersion: grafana.integreatly.org/v1beta1
 kind: GrafanaDashboard
 metadata:
   labels:
-    app.kubernetes.io/component: grafana-dashboard
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: dashboard
   name: k8sgpt-overview
   namespace: {{ .Values.grafanaDashboard.namespace | default (include "k8sgpt-operator.namespace" .) }}
 spec:

chart/operator/templates/k8sgpt-cluster-role-binding.yaml

@@ -4,6 +4,7 @@ metadata:
   name: {{ include "chart.fullname" . }}-k8sgpt
   labels:
     {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: rbac
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole

chart/operator/templates/k8sgpt-cluster-role.yaml

@@ -4,6 +4,7 @@ metadata:
   name: {{ include "chart.fullname" . }}-k8sgpt
   labels:
     {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: rbac
 rules:
   - apiGroups:
       - '*'

chart/operator/templates/k8sgpt-sa.yaml

@@ -4,12 +4,9 @@ kind: ServiceAccount
 metadata:
   name:  {{ default "k8sgpt" .Values.serviceAccount.name }}
   labels:
-    app.kubernetes.io/component: rbac
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
     {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: rbac
   {{- if .Values.serviceAccount.annotations }}
   annotations: {{ toYaml .Values.serviceAccount.annotations | nindent 4 }}
   {{- end }}
 {{- end }}
-

chart/operator/templates/leader-election-rbac.yaml

@@ -3,10 +3,8 @@ kind: Role
 metadata:
   name: {{ include "chart.fullname" . }}-leader-election-role
   labels:
+    {{- include "chart.labels" . | nindent 4 }}
     app.kubernetes.io/component: rbac
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
 rules:
 - apiGroups:
   - ""
@@ -45,15 +43,13 @@ kind: RoleBinding
 metadata:
   name: {{ include "chart.fullname" . }}-leader-election-rolebinding
   labels:
+    {{- include "chart.labels" . | nindent 4 }}
     app.kubernetes.io/component: rbac
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: Role
   name: '{{ include "chart.fullname" . }}-leader-election-role'
 subjects:
 - kind: ServiceAccount
   name: '{{ include "chart.fullname" . }}-controller-manager'
-  namespace: '{{ .Release.Namespace }}'
+  namespace: '{{ .Release.Namespace }}'

chart/operator/templates/manager-rbac.yaml

@@ -3,7 +3,8 @@ kind: ClusterRole
 metadata:
   name: {{ include "chart.fullname" . }}-manager-role
   labels:
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: manager-rbac
 rules:
 - apiGroups:
   - '*'
@@ -67,10 +68,8 @@ kind: ClusterRoleBinding
 metadata:
   name: {{ include "chart.fullname" . }}-manager-rolebinding
   labels:
-    app.kubernetes.io/component: rbac
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: manager-rbac
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole

chart/operator/templates/metrics-reader-rbac.yaml

@@ -3,12 +3,10 @@ kind: ClusterRole
 metadata:
   name: {{ include "chart.fullname" . }}-metrics-reader
   labels:
-    app.kubernetes.io/component: kube-rbac-proxy
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: metrics-reader
 rules:
 - nonResourceURLs:
   - /metrics
   verbs:
-  - get
+  - get

chart/operator/templates/metrics-service.yaml

@@ -3,15 +3,13 @@ kind: Service
 metadata:
   name: {{ include "chart.fullname" . | trunc 20}}-controller-manager-metrics-service
   labels:
-    app.kubernetes.io/component: kube-rbac-proxy
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
     control-plane: controller-manager
-  {{- include "chart.labels" . | nindent 4 }}
+    {{- include "chart.labels" . | nindent 4 }}
+    app.kubernetes.io/component: metrics-reader
 spec:
   type: {{ .Values.metricsService.type }}
   selector:
     control-plane: controller-manager
-  {{- include "chart.selectorLabels" . | nindent 4 }}
+    {{- include "chart.selectorLabels" . | nindent 4 }}
   ports:
-	{{- .Values.metricsService.ports | toYaml | nindent 2 -}}
+	{{- .Values.metricsService.ports | toYaml | nindent 2 -}}

chart/operator/templates/proxy-rbac.yaml

@@ -3,10 +3,8 @@ kind: ClusterRole
 metadata:
   name: {{ include "chart.fullname" . }}-proxy-role
   labels:
+    {{- include "chart.labels" . | nindent 4 }}
     app.kubernetes.io/component: kube-rbac-proxy
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
 rules:
 - apiGroups:
   - authentication.k8s.io
@@ -26,15 +24,13 @@ kind: ClusterRoleBinding
 metadata:
   name: {{ include "chart.fullname" . }}-proxy-rolebinding
   labels:
+    {{- include "chart.labels" . | nindent 4 }}
     app.kubernetes.io/component: kube-rbac-proxy
-    app.kubernetes.io/created-by: k8sgpt-operator
-    app.kubernetes.io/part-of: k8sgpt-operator
-  {{- include "chart.labels" . | nindent 4 }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole
   name: '{{ include "chart.fullname" . }}-proxy-role'
 subjects:
 - kind: ServiceAccount
   name: '{{ include "chart.fullname" . }}-controller-manager'
-  namespace: '{{ .Release.Namespace }}'
+  namespace: '{{ .Release.Namespace }}'

chart/operator/values.yaml

@@ -7,6 +7,9 @@ serviceAccount:
   name: "k8sgpt"
   # -- Annotations for the managed k8sgpt workload service account
   annotations: {}
+# labels that'll be applied to all resources
+additionalLabels: {}
+  # app: k8s-operator
 serviceMonitor:
   enabled: false
   additionalLabels: {}