-
Notifications
You must be signed in to change notification settings - Fork 231
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
Refining the fluent-operator chart #526
Conversation
f56503c
to
e8150f2
Compare
README.md
Outdated
|
||
You can also deploy the Kubernetes logging stack with Helm, just need to set the `Kubernetes` parameter to `ture`: | ||
You can also deploy the Kubernetes logging pipeline with Helm, just need to set the `Kubernetes` parameter to `true`: |
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.
You can also deploy the Kubernetes logging pipeline with Helm, just need to set the `Kubernetes` parameter to `true`: | |
You can also deploy the Kubernetes logging pipeline with Helm, just need to set the `Kubernetes` parameter to `true` (default): |
charts/fluent-operator/values.yaml
Outdated
requests: | ||
cpu: 100m | ||
memory: 20Mi | ||
resources: {} |
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.
resources is still empty?
#Configure the output plugin parameter in Fluentd. | ||
#You can set enable to true to output logs to the specified location. | ||
resources: | ||
limits: |
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.
fluentd limits is too small
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.
200m and 200Mi is still too small
charts/fluent-operator/values.yaml
Outdated
@@ -92,7 +100,7 @@ fluentbit: | |||
tail: | |||
memBufLimit: 5MB | |||
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics | |||
# nodeExporterMetrics: | |||
nodeExporterMetrics: |
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.
Why enable nodeExporterMetrics by default?
6838e5a
to
9831e97
Compare
charts/fluent-operator/values.yaml
Outdated
@@ -92,7 +100,7 @@ fluentbit: | |||
tail: | |||
memBufLimit: 5MB | |||
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics |
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.
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics | |
nodeExporterMetrics: {} | |
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics |
charts/fluent-operator/values.yaml
Outdated
@@ -92,7 +100,7 @@ fluentbit: | |||
tail: | |||
memBufLimit: 5MB | |||
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics |
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.
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics | |
# uncomment below nodeExporterMetrics section if you want to collect node exporter metrics |
690dda4
to
db0f476
Compare
Signed-off-by: chengdehao <dehaocheng@kubesphere.io>
Signed-off-by: chengdehao dehaocheng@kubesphere.io
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduced a user-facing change?
Additional documentation, usage docs, etc.: