Conversation
workaround is to turn off daily backup Signed-off-by: Steve Root <steve@rkbb.co.uk>
Add note about long borg restore clashing with new backups
|
I'm new to this, I'm not sure how to add labels. |
pabzm
added
3. to review
pabzm
left a comment
pabzm
left a comment
There was a problem hiding this comment.
Thank you very much for the contribution, it definitely is useful!
I've got two comments, please have a look at them.
| --- | ||
| ### Restoring a borg backup ### | ||
| Beware that a long running restore process may be interuprted by the start of a new backup process. | ||
| See [https://github.com/nextcloud/all-in-one/blob/main/docker-ipv6-support.md](https://github.com/nextcloud/all-in-one/issues/8414)) |
There was a problem hiding this comment.
Showing one URL but linking to another smells a little bit strange. Is that on purpose? If so, why?
There was a problem hiding this comment.
Not on purpose, that would be me struggling to work out how to do this :-)
Thanks for spotting, leave it with me and I'll resubmit and learn from doing it.
Thanks for reviewing
|
|
||
| --- | ||
| ### Restoring a borg backup ### | ||
| Beware that a long running restore process may be interuprted by the start of a new backup process. |
There was a problem hiding this comment.
There's a typo here:
| Beware that a long running restore process may be interuprted by the start of a new backup process. | |
| Beware that a long running restore process may be interrupted by the start of a new backup process. |
szaimen
left a comment
szaimen
left a comment
There was a problem hiding this comment.
Instead of documenting the workaround we should rather try to fix the underlying problem imho.
szaimen
added
2. developing
|
I'm on holiday next week so plan to fix the pull request and resubmit. |
All good! There is an open PR for this now here: #8514 |
This branch has not been deployed
3 participants
Long running restores can be interrupted by new backups starting.
Summary
3. to review, feature component)AI (if applicable)