-
Notifications
You must be signed in to change notification settings - Fork 183
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
docs: add fluentd buffers vs DPM calculations info for metrics #2128
Conversation
1dbdc16
to
3a4420d
Compare
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.
This lgTM. Do we not want to have a section for logs/events as well?
I think it'd be safer to divide the final values by 2 to account for uneven load on FluentD instances. Even with the remote write proxy, we're still relying on Kubernetes Service load balancing, which is far from perfect. |
9658197
to
7c457c1
Compare
Fair point, I've added that info and recommendation. |
We didn't check that for logs or events so cannot give such info yet. |
2473f65
to
06f3030
Compare
06f3030
to
d2631b0
Compare
Description
Fill in your description here.
Checklist
Remove items which don't apply to your PR.
Testing performed