Simplified adding labels and only use selector labels where appropriate

minio · Apr 16, 2024 · 9c2b261 · 9c2b261
1 parent bf5638d
commit 9c2b261
Show file tree

Hide file tree

Showing 14 changed files with 29 additions and 41 deletions.
diff --git a/helm/operator/templates/_helpers.tpl b/helm/operator/templates/_helpers.tpl
@@ -37,25 +37,17 @@ Create chart name and version as used by the chart label.
 {{- end -}}
 
 {{/*
-Common labels for operator. Does not include selectorLabels.
+Common labels for operator
 */}}
-{{- define "minio-operator.labels-common" -}}
+{{- define "minio-operator.labels" -}}
 helm.sh/chart: {{ include "minio-operator.chart" . }}
-{{- range $key, $val :=  .Values.operator.additionalLabels }}
-{{ $key }}: {{ $val | quote }}
-{{- end }}
 {{- if .Chart.AppVersion }}
 app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 {{- end }}
 app.kubernetes.io/managed-by: {{ .Release.Service }}
-{{- end -}}
-
-{{/*
-All labels for operator. Includes selectorLabels.
-*/}}
-{{- define "minio-operator.labels-all" -}}
-{{ include "minio-operator.labels-common" . }}
-{{ include "minio-operator.selectorLabels" . }}
+{{- range $key, $val :=  .Values.operator.additionalLabels }}
+{{ $key }}: {{ $val | quote }}
+{{- end }}
 {{- end -}}
 
 {{/*
@@ -67,25 +59,17 @@ app.kubernetes.io/instance: {{ .Release.Name }}
 {{- end -}}
 
 {{/*
-Common labels for console. Does not include selectorLabels.
+Common labels for console
 */}}
-{{- define "minio-operator.console-labels-common" -}}
+{{- define "minio-operator.console-labels" -}}
 helm.sh/chart: {{ include "minio-operator.chart" . }}
-{{- range $key, $val := .Values.console.additionalLabels }}
-{{ $key }}: {{ $val | quote }}
-{{- end }}
 {{- if .Chart.AppVersion }}
 app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 {{- end }}
 app.kubernetes.io/managed-by: {{ .Release.Service }}
-{{- end -}}
-
-{{/*
-All labels for console. Combines minio-operator.console-labels-common and minio-operator.console-selectorLabels.
-*/}}
-{{- define "minio-operator.console-labels-all" -}}
-{{ include "minio-operator.console-labels-common" . }}
-{{ include "minio-operator.console-selectorLabels" . }}
+{{- range $key, $val := .Values.console.additionalLabels }}
+{{ $key }}: {{ $val | quote }}
+{{- end }}
 {{- end -}}
 
 {{/*

diff --git a/helm/operator/templates/console-clusterrole.yaml b/helm/operator/templates/console-clusterrole.yaml
@@ -3,7 +3,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata:
   name: console-sa-role
-  labels: {{- include "minio-operator.console-labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 rules:
   - apiGroups:
       - ""

diff --git a/helm/operator/templates/console-clusterrolebinding.yaml b/helm/operator/templates/console-clusterrolebinding.yaml
@@ -3,7 +3,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRoleBinding
 metadata:
   name: console-sa-binding
-  labels: {{- include "minio-operator.console-labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole

diff --git a/helm/operator/templates/console-configmap.yaml b/helm/operator/templates/console-configmap.yaml
@@ -4,7 +4,7 @@ kind: ConfigMap
 metadata:
   name: console-env
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.console-labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 data:
   CONSOLE_PORT: "9090"
   CONSOLE_TLS_PORT: "9443"

diff --git a/helm/operator/templates/console-deployment.yaml b/helm/operator/templates/console-deployment.yaml
@@ -4,14 +4,16 @@ kind: Deployment
 metadata:
   name: console
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.console-labels-all" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 spec:
   replicas: {{ .Values.console.replicaCount }}
   selector:
     matchLabels: {{- include "minio-operator.console-selectorLabels" . | nindent 6 }}
   template:
     metadata:
-      labels: {{- include "minio-operator.console-labels-all" . | nindent 8 }}
+      labels:
+        {{- include "minio-operator.console-labels" . | nindent 8 }}
+        {{- include "minio-operator.console-selectorLabels" . | nindent 8 }}
     spec:
       {{- with .Values.console.imagePullSecrets }}
       imagePullSecrets: {{- toYaml . | nindent 8 }}

diff --git a/helm/operator/templates/console-secret.yaml b/helm/operator/templates/console-secret.yaml
@@ -6,6 +6,6 @@ metadata:
   namespace: {{ .Release.Namespace }}
   annotations:
     kubernetes.io/service-account.name: console-sa
-  labels: {{- include "minio-operator.console-labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 type: kubernetes.io/service-account-token
 {{- end }}
diff --git a/helm/operator/templates/console-service.yaml b/helm/operator/templates/console-service.yaml
@@ -4,7 +4,7 @@ kind: Service
 metadata:
   name: console
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.console-labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 spec:
   ports:
   - name: http

diff --git a/helm/operator/templates/console-serviceaccount.yaml b/helm/operator/templates/console-serviceaccount.yaml
@@ -4,5 +4,5 @@ kind: ServiceAccount
 metadata:
   name: console-sa
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.console-labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.console-labels" . | nindent 4 }}
 {{- end }}
diff --git a/helm/operator/templates/operator-clusterrole.yaml b/helm/operator/templates/operator-clusterrole.yaml
@@ -2,7 +2,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata:
   name: minio-operator-role
-  labels: {{- include "minio-operator.labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.labels" . | nindent 4 }}
 rules:
   - apiGroups:
       - "apiextensions.k8s.io"

diff --git a/helm/operator/templates/operator-clusterrolebinding.yaml b/helm/operator/templates/operator-clusterrolebinding.yaml
@@ -2,7 +2,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRoleBinding
 metadata:
   name: minio-operator-binding
-  labels: {{- include "minio-operator.labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.labels" . | nindent 4 }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole

diff --git a/helm/operator/templates/operator-deployment.yaml b/helm/operator/templates/operator-deployment.yaml
@@ -3,14 +3,16 @@ kind: Deployment
 metadata:
   name: minio-operator
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.labels-all" . | nindent 4 }}
+  labels: {{- include "minio-operator.labels" . | nindent 4 }}
 spec:
   replicas: {{ .Values.operator.replicaCount }}
   selector:
     matchLabels: {{- include "minio-operator.selectorLabels" . | nindent 6 }}
   template:
     metadata:
-      labels: {{- include "minio-operator.labels-all" . | nindent 8 }}
+      labels:
+        {{- include "minio-operator.labels" . | nindent 8 }}
+        {{- include "minio-operator.selectorLabels" . | nindent 8 }}
     spec:
       {{- with .Values.operator.imagePullSecrets }}
       imagePullSecrets: {{- toYaml . | nindent 8 }}

diff --git a/helm/operator/templates/operator-service.yaml b/helm/operator/templates/operator-service.yaml
@@ -3,7 +3,7 @@ kind: Service
 metadata:
   name: operator
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.labels" . | nindent 4 }}
 spec:
   type: ClusterIP
   ports:

diff --git a/helm/operator/templates/operator-serviceaccount.yaml b/helm/operator/templates/operator-serviceaccount.yaml
@@ -3,7 +3,7 @@ kind: ServiceAccount
 metadata:
   name: minio-operator
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.labels" . | nindent 4 }}
   {{- with .Values.operator.serviceAccountAnnotations }}
   annotations:
     {{- toYaml . | nindent 4 }}

diff --git a/helm/operator/templates/sts-service.yaml b/helm/operator/templates/sts-service.yaml
@@ -3,7 +3,7 @@ kind: Service
 metadata:
   name: sts
   namespace: {{ .Release.Namespace }}
-  labels: {{- include "minio-operator.labels-common" . | nindent 4 }}
+  labels: {{- include "minio-operator.labels" . | nindent 4 }}
 spec:
   type: ClusterIP
   ports: