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: optionally use standalone mode for syncing using Docker container #3018

Merged
merged 3 commits into from
Nov 28, 2024

Conversation

PieterScheffers
Copy link
Contributor

I am using the Docker container to sync the files from a Onedrive account to my NAS. There I take a ZFS snapshot of the files every night. The Docker container currently only supports monitor mode.

It would be better if the sync only runs once in the night via a cronjob. When the sync is finished the ZFS snapshot can be taken. This saves a lot of CPU cycles during the day and enables the snapshot to be taken after the sync has finished. Also when the sync would error it would return a non-zero exitcode. This enables error to be send by the error reporting of the cronjobs.

This comment has been minimized.

This comment has been minimized.

* Update podman.md as well
Copy link

@check-spelling-bot Report

🔴 Please review

See the 📂 files view, the 📜action log, or 📝 job summary for details.

Unrecognized words (41)
bashisms
capitalisation
containerised
customisable
customisation
cuz
finalise
finalised
fullstop
gzipping
initialising
Ioctl
ISPs
kinda
Kubuntu
logfiles
Lubuntu
malware
meh
optimisations
preinstalled
prioritisation
prioritise
prolly
reparse
resynchronisation
sandboxing
segfault
shitload
Slackware
sourced
Sourcing
spamming
thingies
Thu
uninitialised
uninstallation
upgradable
utilisation
websites
Xubuntu
To accept these unrecognized words as correct, you could run the following commands

... in a clone of the [email protected]:PieterScheffers/onedrive.git repository
on the sync_once_docker_option branch (ℹ️ how do I use this?):

curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/abraunegg/onedrive/actions/runs/12074553655/attempts/1'

@abraunegg
Copy link
Owner

@PieterScheffers
Thanks for contributing - will merge this into 'master' , then update your PR

@abraunegg abraunegg merged commit c7f1b7f into abraunegg:master Nov 28, 2024
3 of 4 checks passed
@PieterScheffers
Copy link
Contributor Author

@PieterScheffers Thanks for contributing - will merge this into 'master' , then update your PR

Thanks for merging this!

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