-
Notifications
You must be signed in to change notification settings - Fork 79
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
Add throttling metrics #227
Conversation
Signed-off-by: Athish Pranav D <athishanna@gmail.com>
Signed-off-by: Athish Pranav D <athishanna@gmail.com>
@ashie this is dependent on fluent/fluentd#4578 Like the above PR is merged, but whoever didn't update Fluentd will have error if they use this code (since the metrics reference is dangling). What's your suggestion for it? |
Please check existence of |
Signed-off-by: Athish Pranav D <athishanna@gmail.com>
@ashie I will add the manual testing info once i get time |
@ashie any other checks do you want me to do? |
Thanks! |
Issues this PR solves
#190
Why is this PR needed?
Add observability on throttling in
in_tail
pluginfluent/fluentd#4578 - corresponding metrics PR in
in_tail