Skip to content

Commit 4e7b751

Browse files
committed
changes per peer review comments
1 parent aeb9cc3 commit 4e7b751

File tree

3 files changed

+3
-12
lines changed

3 files changed

+3
-12
lines changed

charts/fluent-bit/templates/daemonset.yaml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,7 @@ spec:
2727
template:
2828
metadata:
2929
labels:
30-
{{- if .Values.verbosePodLabels.enabled }}
31-
{{- include "fluent-bit.verbosePodLabels" . | nindent 8 }}
32-
{{- else }}
33-
{{- include "fluent-bit.selectorLabels" . | nindent 8 }}
34-
{{- end }}
30+
{{- include (ternary "fluent-bit.verbosePodLabels" "fluent-bit.selectorLabels" .Values.verbosePodLabels) . | nindent 8 }}
3531
{{- with .Values.podLabels }}
3632
{{- toYaml . | nindent 8 }}
3733
{{- end }}

charts/fluent-bit/templates/deployment.yaml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,7 @@ spec:
3030
template:
3131
metadata:
3232
labels:
33-
{{- if .Values.verbosePodLabels.enabled }}
34-
{{- include "fluent-bit.verbosePodLabels" . | nindent 8 }}
35-
{{- else }}
36-
{{- include "fluent-bit.selectorLabels" . | nindent 8 }}
37-
{{- end }}
33+
{{- include (ternary "fluent-bit.verbosePodLabels" "fluent-bit.selectorLabels" .Values.verbosePodLabels) . | nindent 8 }}
3834
{{- with .Values.podLabels }}
3935
{{- toYaml . | nindent 8 }}
4036
{{- end }}

charts/fluent-bit/values.yaml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -303,8 +303,7 @@ podAnnotations: {}
303303
podLabels: {}
304304

305305
# adds app.kubernetes.io/version label to pods for istio required labels
306-
verbosePodLabels:
307-
enabled: false
306+
verbosePodLabels: false
308307

309308
## How long (in seconds) a pods needs to be stable before progressing the deployment
310309
##

0 commit comments

Comments
 (0)