From 392865ec594c796fc90d77fb963097287c116a00 Mon Sep 17 00:00:00 2001 From: Marcus Rodan Date: Wed, 28 Apr 2021 10:56:16 +0200 Subject: [PATCH] Fix Deployment of Flagger loadtester to include the correct label The Deployment of the Flagger loadtester did not contain the correct label app.kubernetes.io/name. This label is used for the Flagger deployment and it is also used in the PodDisruptionBudget for the Flagger operator. I added the same label to the Flagger load tester to make the PodDisruptionBudget work correctly for the Flagger loadtester. Signed-off-by: Marcus Rodan --- charts/loadtester/templates/deployment.yaml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/charts/loadtester/templates/deployment.yaml b/charts/loadtester/templates/deployment.yaml index 7236a288..f0a1bf7d 100644 --- a/charts/loadtester/templates/deployment.yaml +++ b/charts/loadtester/templates/deployment.yaml @@ -16,6 +16,7 @@ spec: metadata: labels: app: {{ include "loadtester.name" . }} + app.kubernetes.io/name: {{ include "loadtester.name" . }} annotations: appmesh.k8s.aws/ports: "444" {{- if .Values.podAnnotations }} @@ -29,7 +30,7 @@ spec: {{- end }} {{- if .Values.podPriorityClassName }} priorityClassName: {{ .Values.podPriorityClassName }} - {{- end }} + {{- end }} containers: - name: {{ .Chart.Name }} {{- if .Values.securityContext.enabled }} @@ -69,7 +70,7 @@ spec: {{- if .Values.env }} env: {{- toYaml .Values.env | nindent 12 }} - {{- end }} + {{- end }} resources: {{- toYaml .Values.resources | nindent 12 }} {{- with .Values.nodeSelector }}