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

[svlogd] prevent deletion of the processing file #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stieg
Copy link

@stieg stieg commented Mar 29, 2024

svlogd expects time to be correct, but sometimes this is not the case. If time information is inaccurate, it is possible for svlogd to delete the file it is about to process. When this happens, svlogd will enter an infinite busy loop trying to process a file that no longer exists. This patch prevents that case from happening by excluding processing file from consideration when removing old log files.

svlogd expects time to be correct, but sometimes this is not the case. If time
information is inaccurate, it is possible for svlogd to delete the file it is
about to process. When this happens, svlogd will enter an infinite busy loop
trying to process a file that no longer exists. This patch prevents that case
from happening by excluding processing file from consideration when removing old
log files.
@stieg
Copy link
Author

stieg commented Apr 4, 2024

image

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

Successfully merging this pull request may close these issues.

1 participant