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
helm support #204
helm support #204
Conversation
initContainers: | ||
- name: setenv | ||
image: {{ .Values.operator.initcontainer.image }}:{{ .Values.operator.initcontainer.tag }} | ||
command: | ||
- /bin/sh | ||
- '-c' | ||
- set -ex; | ||
echo CONTAINER_ROOT_DIR=$(crictl info --output go-template --template '{{`{{.config.containerdRootDir}}`}}') > /fluentbit-operator/fluent-bit.env | ||
echo CONTAINER_ROOT_DIR={{ .Values.logPath.containerd }} > /fluent-operator/fluent-bit.env |
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.
Refer to the changes in latest PR ac62edc#diff-11a3a526accdc2c72ddd4b8cf745ad478e23fa61a1ad80dd62d532e47cae40a7R33
@zhu733756 please have a discussion with @wenchajun on how to change this part
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.
fixed. This is used because it is the field that is the containerd
or crio
field. For docker
environments, this field has been modified.
22ab064
to
3f64144
Compare
charts/fluent-operator/templates/fluentd-output-elasticsearch.yaml
Outdated
Show resolved
Hide resolved
enable: false | ||
brokers: "<kafka broker list like xxx.xxx.xxx.xxx:9092,yyy.yyy.yyy.yyy:9092>" | ||
fluentbit: | ||
forward: |
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.
fluentbit also acts as a output layer to send to kafka or es. Pls fill back them.
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.
Thanks. Fixed.
Signed-off-by: chengdehao <dehaocheng@kubesphere.io>
Signed-off-by: chengdehao dehaocheng@kubesphere.io