-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Feature request: Use last modified files when log limit is hit. #2748
Comments
Hi @jannylund, thanks for reporting this. We are having a look and will reach back in the next few days |
@tmichelet is my understanding correct that the files are temporarily opened, tailed and then closed? Or will they be permanently kept open? I'm thinking I could make a shell script to symlink them to a log folder as a quick workaround. |
@jannylund they are permanently kept open (only closed on logrotate or when stopping the tailer) |
I made a workaround in the meanwhile.
This causes the DD Agent to open up tails on the symlink added event and close them on delete event. |
@jannylund this workaround should do the job indeed! On our end, we're queuing some work to properly tackle this issue. Thanks again for reporting |
Hi @jannylund, do you have the name format of your files ? And do you have control on the name of those files ? Getting the ordered list of files by last modified date for 50k+ files is quite resource intensive and might seriously increase the agent footprint on its host. We're investigating a more gentle way of doing it based on the filenames. Sharing a tree of your folder would be helpful Thank you! |
We just implemented this feature in #2796, it will be released in 6.10 end of February! |
Describe what happened:
/bamboo_home/xml-data/builds/*/download-data/build_logs/*.log
2018-12-06 19:48:26 UTC | WARN | (file_provider.go:98 in FilesToTail) | Reached the limit on the maximum number of files in use: 100
Describe what you expected:
Suggestion:
Modify the method FilesToTail to order files by last modified instead of alphabetical.
The text was updated successfully, but these errors were encountered: