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

syncthing: Update to 1.29.2 and Change logfile path #25790

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

Conversation

AutoCONFIG
Copy link

@AutoCONFIG AutoCONFIG commented Jan 19, 2025

Update to 1.29.2 and Change logfile path from /etc/syncthing/syncthing log to /var/log/syncthing.log.
This utilizes tmpfs (memory-based storage) to prevent disk usage growth caused by log file accumulation during extended operation.

Benefits:

  • Avoids storage saturation from excessive log growth.
  • Aligns with best practices for temporary log management.
  • For systems using flash storage (e.g., SSDs or eMMC), storing logs in tmpfs avoids flash memory wear caused by frequent writes to /etc/syncthing.

Maintainer: Paul Spooren [email protected], Van Waholtz [email protected]
Compile tested: x86-64(Ubuntu 24), master(24.10-rc5)
Run tested: x86-64, master(24.10-rc5), it's running normally.
Description:
The current version is two years old, and the default log path is under /etc/syncthing, which does not automatically clean up after restarting the machine or program, resulting in wasted storage space (especially for small embedded devices).

Because the repository was forked again, the previous PR cannot be reopened.
Regarding the previously mentioned make -j compilation error, it seems that it's due to the lack of support for make -j compilation when building Go in the OpenWRT SDK, and it doesn't affect the normal source code compilation process. The rest of the test results are all normal.

@1715173329
Copy link
Member

cc @brvphoenix

@1715173329
Copy link
Member

please update your git author config:
git config --local user.name "Yun Wang"
then update the commit:
git commit --amend --reset-author --no-edit && git push --force

Update to 1.29.2 and Change logfile path from /etc/syncthing/syncthing
log to /var/log/syncthing.log.
This utilizes tmpfs (memory-based storage) to prevent disk usage growth
caused by log file accumulation during extended operation.

Benefits:
- Avoids storage saturation from excessive log growth.
- Aligns with best practices for temporary log management.
- For systems using flash storage (e.g., SSDs or eMMC), storing logs in
  tmpfs avoids flash memory wear caused by frequent writes to
  /etc/syncthing.

Signed-off-by: Yun Wang <[email protected]>
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