From 380eb06ae94c1b34b4914cd25c6f49be939eaa6c Mon Sep 17 00:00:00 2001 From: Niladri Halder Date: Wed, 18 Dec 2024 20:37:20 +0530 Subject: [PATCH] fix(chart): move app=componentName label out of csi-node matchLabels The label `app: openebs-zfs-node` was missing on the csi-node components. However, adding it to the selector labels section breaks upgrade from previous versions. This need not live in matchLabels, and could simply live in componentLabels Signed-off-by: Niladri Halder --- deploy/helm/charts/templates/_helpers.tpl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deploy/helm/charts/templates/_helpers.tpl b/deploy/helm/charts/templates/_helpers.tpl index 9231ad174..834f1418e 100644 --- a/deploy/helm/charts/templates/_helpers.tpl +++ b/deploy/helm/charts/templates/_helpers.tpl @@ -98,7 +98,6 @@ Create labels for openebs zfs-localpv controller Create match labels for openebs zfs-localpv node daemon */}} {{- define "zfslocalpv.zfsNode.matchLabels" -}} -app: {{ .Values.zfsNode.componentName | quote }} name: {{ .Values.zfsNode.componentName | quote }}{{ if or (not (hasKey .Values "enableHelmMetaLabels")) .Values.enableHelmMetaLabels }} release: {{ .Release.Name }} {{- end -}} @@ -108,6 +107,7 @@ release: {{ .Release.Name }} Create component labels openebs zfs-localpv node daemon */}} {{- define "zfslocalpv.zfsNode.componentLabels" -}} +app: {{ .Values.zfsNode.componentName | quote }} openebs.io/component-name: {{ .Values.zfsNode.componentName | quote }} {{- end -}}