-
Notifications
You must be signed in to change notification settings - Fork 573
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 remote borg backup support #4804
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
I.e. do not support aio-lockfile for remote borg repos. Let's wait and see if if somebody requests it. Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
4d3e48c
to
7e96d1d
Compare
Hi, first of all thank you for your contribution! I've thought a bit about this the last week and came to the conclusion that I do not want to maintain this since it requires an additional server for me for testing (with a not so easy to reproduce setup) and has the potential of many users needing help getting this to work (and thus only a limited usecase). So I would still only test local backup also in the future. However I would be fine with merging this if you @timdiels would step in as the maintainer of this feature. That means if bug reports or any questions regarding this feature come in, I would ask you for help on the topic. If that is fine for you, I would continue with the review.
Yes, this is expected and documented.
Sounds good!
Yeah, we could create a dev instance from this if we proceed...
Yes |
if ! rsync --archive --human-readable -vv \ | ||
/tmp/borg/nextcloud_aio_volumes/nextcloud_aio_mastercontainer/data/configuration.json \ | ||
/nextcloud_aio_volumes/nextcloud_aio_mastercontainer/data/configuration.json; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would personally like to keep rsync at least for local archives since as you found out borg extract does not allow for deleting existing files and rsync is much faster at least for local archives... but we could split the logic and use rsync for local archives and borg extract for remote archives...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I'll bring back the borg mount and rsync for local repos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Hi,
I'm willing to maintain it, mostly on weekends, for as long as I'm a nextcloud user (which I have been for a couple of years so far).
I will do it on the dev build after I've made the change you suggested.
Is there an easier way to develop than the hacks I did (in a different branch) for local testing? Not sure how you normally test AIO? If it's always via push to github, wait for build and deploy, I will probably keep my debug branch for later. Note to self:
|
Cool, then we can go ahead with this PR. I've invited you to the repo for easier collaboration :)
Great :)
Usually it is indeed always via push to github, wait for build and deploy. So probably it is easier for you to keep your debug branch 👍 |
Very much appreciate the PR guys! |
echo "Could not initialize borg repository." | ||
rm -f "$BORG_BACKUP_DIRECTORY/config" | ||
if [ -z "$BORG_REMOTE_REPO" ]; then | ||
# Originally we checked for presence of the config file instead of calling `borg info`. Likely `borg info` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be "Likely borg init
will error on a partially initialized repo"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably most borg commands would fail but it's referring to the borg info on line 120. If we ever end up with a partially initialized repo (sounds very rare), I think manual intervention is required anyway? The important thing is I believe borg info
will fail on such a corrupt repo and will end up at line 124 instead of trying to initialize it again. The config file was used as a sentinel to detect whether the repo has been initialized already (see the deleted line 107), which why it is removed here and why it's probably no longer necessary to do this.
Signed-off-by: Tim Diels <[email protected]>
Signed-off-by: Tim Diels <[email protected]>
I tried out local backups on my machine and made all the requested changes I believe. Could we get a a dev build going? |
Signed-off-by: Simon L. <[email protected]>
@@ -66,7 +66,7 @@ RUN set -ex; \ | |||
wget https://getcomposer.org/installer -O - | php -- --install-dir=/usr/local/bin --filename=composer; \ | |||
chmod +x /usr/local/bin/composer; \ | |||
cd /var/www/docker-aio; \ | |||
git clone https://github.com/nextcloud-releases/all-in-one.git --depth 1 .; \ | |||
git clone https://github.com/timdiels/nextcloud-aio.git --depth 1 --branch feature/remote-borg-backup3 .; \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: revert this later on
Nice!
Yes, it should be available via |
I'm trying it out on my own server (with a VM snapshot before switching to dev). Will I be able to later update this to |
This fixes: borg config: error: argument REPOSITORY: "::": Repository must be local Signed-off-by: Tim Diels <[email protected]>
Created a backup on my server, I'll enable autobackup/update and try out restoring after some changes later (might be next weekend):
(the duration is pretty much optimal considering my 19 Mbps ISP upload speed) |
I rolled back to an old snapshot because apparently the dev build was older than the latest released nextcloud, so it wouldn't start anymore because e.g. elastic can't roll back. I'll try again on the weekend with a tmp vm instead. |
Appreciate all the work you are doing on this @timdiels. Very much looking forward to this addition! |
This adds support for backing up directly to a remote borg repo. (I tried reaching out earlier)
Why?
I'd like to backup/restore via the UI, resist ransomware and store the backups remotely without storing a local (compressed) copy of the nextcloud data.
Alternatives (https://github.com/nextcloud/all-in-one?tab=readme-ov-file#are-remote-borg-backups-supported):
How to use it
Instead of entering a local repo path (which remains supported), you can choose to instead enter a remote borg repo url. (These screenshots are from a disaster recovery but it's analogous, just ignore the passphrase):
The first you try to initialise the repo, the backup container will create an ssh key and foolishly try to init the borg repo resulting in error because you have to authorise the ssh key it generated first. So, e.g. I copy paste the public key shown here to my borgbase repo:
Then I can try again as instructed and it should work. The following backup info is shown:
Changes
borg extract
because, at least with a remote repo,borg mount
is very slow (20 seconds vs 24 minutes restore for pretty much an empty nextcloud install).FYI Oddly the original code never restores host-mounts.
Testing
I hacked the code til I could run local deployments of nextcloud-aio without any builds and manually tested:
TODO