-
Notifications
You must be signed in to change notification settings - Fork 229
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
fluent operator & fluentbit: Added tolerations, nodeSelector + more #704
Conversation
Signed-off-by: flxman <felix.farjsjo@gmail.com>
Signed-off-by: illrill <richard.jimmy.johansson@gmail.com>
imagePullSecrets: {{ toYaml .Values.operator.imagePullSecrets | nindent 8 }} | ||
{{- end }} | ||
{{- if .Values.operator.tolerations }} |
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.
maybe
imagePullSecrets:
{{ toYaml .Values.operator.imagePullSecrets | nindent 8 }}
{{- end }}
better
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.
Indentation style is now aligned with the rest of the manifest.
imagePullSecrets: {{ toYaml .Values.operator.imagePullSecrets | nindent 8 }} | ||
{{- end }} | ||
{{- if .Values.operator.tolerations }} | ||
tolerations: {{ toYaml .Values.operator.tolerations | nindent 8 }} |
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.
ditto
tolerations: {{ toYaml .Values.operator.tolerations | nindent 8 }} | ||
{{- end }} | ||
{{- if .Values.operator.nodeSelector }} | ||
nodeSelector: {{ toYaml .Values.operator.nodeSelector | nindent 8 }} |
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.
ditto
nodeSelector: {{ toYaml .Values.operator.nodeSelector | nindent 8 }} | ||
{{- end }} | ||
{{- if .Values.operator.podSecurityContext }} | ||
podSecurityContext: {{ toYaml .Values.operator.podSecurityContext | nindent 8 }} |
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.
ditto
Signed-off-by: illrill <richard.jimmy.johansson@gmail.com>
@felfa01 Thank you! |
What this PR does / why we need it:
Added support for tolerations, nodeSelector, priorityClassName and podSecurityContext to fluent-operator deployment and fluentbit daemonset
Which issue(s) this PR fixes:
Fixes fluent/helm-charts#314
Fixes #695
Does this PR introduced a user-facing change?
Additional documentation, usage docs, etc.: