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

feat: rotate jmwalletd logs #72

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

feat: rotate jmwalletd logs #72

wants to merge 4 commits into from

Conversation

theborakompanioni
Copy link
Collaborator

@theborakompanioni theborakompanioni commented Oct 24, 2022

TBD.

!! Does not work as expected yet - jmwalletd keeps writing to the rotated file !!

Uses savelog to rotate /var/log/jam/jmwalletd_stdout.log.
Invoked via cron at midnight on every second day (0 0 */2 * *).

Eventually dinit has support for it natively–let's see what comes out of davmac314/dinit#121.

How to test?

TBD.

@theborakompanioni theborakompanioni self-assigned this Oct 24, 2022
@mobin-2008
Copy link
Contributor

I don't know much about this project but Seems like this PR is blocked due to fail to find "a good Solution". I suggest use s6-log. Thats read from stdin and write to file and support automated log rotation without cron. Also it's available on most distributions (such as Alpine, Debian, Ubuntu) via s6 package.

@theborakompanioni
Copy link
Collaborator Author

I don't know much about this project but Seems like this PR is blocked due to fail to find "a good Solution". I suggest use s6-log. Thats read from stdin and write to file and support automated log rotation without cron. Also it's available on most distributions (such as Alpine, Debian, Ubuntu) via s6 package.

Hey @mobin-2008, sorry for the late reply. Thanks so much for following up on this and the valuable information - I did not know about s6-log and will take a look!

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.

2 participants