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

Add timeout to systemd script, stop child units and document absolute and relative folder names #389

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jankatins
Copy link

This is a bit of an overhaul of the systemd based sync:

  • Document that one can use an absolute path as well (no code change,s just the eror message and the readme needed to be adjusted)
  • Add a timeout to the grive run to prevent grive getting stuck and no one is noticing... (this happened freuently on my side and needed a killall grive a lot :-()
  • Instruct the [email protected] unit to stop the timer and changes unit.
  • Ignore some generated files
  • some refactorings and dependency checking (I finally ran a shell linter on this file...) -> This should actually at some point run under set -e -u -o pipefail but I've jut started testing that...

No functional change, just the error message needed to be adjusted.
Also fix some shelllint problems.
This prevents a hanging grive to be stuck for days without syncing the data (which might lead to data loss...).
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