Skip to content
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

Metrics for log throttling per file #190

Closed
mrueg opened this issue Jul 27, 2021 · 8 comments
Closed

Metrics for log throttling per file #190

mrueg opened this issue Jul 27, 2021 · 8 comments

Comments

@mrueg
Copy link
Contributor

mrueg commented Jul 27, 2021

Fluentd 1.13 includes a log throttling per file feature.
https://github.com/fluent/fluentd/blob/master/CHANGELOG.md#enhancement-2

It would be great if there was a way to publish active throttling through metrics to allow early detection.

@Athishpranav2003
Copy link
Contributor

@ashie can i pick this up if its of actual use in the tail monitor?

@ashie
Copy link
Member

ashie commented Aug 1, 2024

@Athishpranav2003
Copy link
Contributor

Oh I see
Fine
Can I work on this one?

@Athishpranav2003
Copy link
Contributor

@ashie for this one which metrics would be useful for us?
the number of lines read from each file ?

@ashie
Copy link
Member

ashie commented Aug 2, 2024

Oh I see Fine Can I work on this one?

Of course 😃

@ashie for this one which metrics would be useful for us? the number of lines read from each file ?

Need to consider but counting number of times throttling is occurred might be down-to-earth approach:
https://github.com/fluent/fluentd/blob/64595e2ac291a3e1d2d02feefdc58e623b3abb02/lib/fluent/plugin/in_tail.rb#L1177

@Athishpranav2003
Copy link
Contributor

Athishpranav2003 commented Aug 4, 2024

@ashie
number of times throttling occured means that how many times we are returning a false whenever the check is failed right? Because when i was taking a look at the code, the count of actual number of lines seems to be not possible to be evaluated since we pause,resume and rotate files.

So i will go ahead and count the number of times we hit the check for limit reached

@Athishpranav2003
Copy link
Contributor

@ashie can we close this issue?

@ashie
Copy link
Member

ashie commented Aug 9, 2024

Fixed via #227
Thanks!

@ashie ashie closed this as completed Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants