-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bitnami/fluent-bit] Fix typo daemonset.enable #31088
Conversation
9c8f644
to
3adfd76
Compare
3adfd76
to
1998d01
Compare
Thanks for your contribution! Could you please bump the chart version in the Chart.yaml? This is necessary to test the changes and cut a new release. |
1998d01
to
72e2561
Compare
Changed |
I just discovered other templates with the same issue in this chart, taking advantage of this PR, would you mind fixing them as well? See $ ag 'daemonset\.enable'
templates/vpa.yaml
35: kind: {{ ternary "DaemonSet" "Deployment" .Values.daemonset.enable }}
templates/pdb.yaml
6:{{- if and .Values.pdb.create (not .Values.daemonset.enable) }}
templates/hpa.yaml
6:{{- if and (not .Values.daemonset.enable) .Values.autoscaling.hpa.enabled }} |
Signed-off-by: Eugene Kim <eugene70kim@gmail.com>
6cfca2b
to
1dcf820
Compare
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Fixed all and changed title. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Alvaro Neira Ayuso <alvaroneay@gmail.com>
Signed-off-by: Alvaro Neira Ayuso <alvaro.neira@broadcom.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
There is a typo in the condition for generating pdb as described in issue #30539. We need to change enable to enabled.
The issue closed, but not solved.