-
Notifications
You must be signed in to change notification settings - Fork 140
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
FR: Notifications for consistency check #1632
Comments
I also wonder what happens technically when the Vorta notification is displayed that the backup is complete and I then remove the external drive before the consistency check has been completed. |
That would make more sense.
@ThomasWaldmann will have to tell you. |
Define "consistency check" in terms of borg commands. |
I think |
Usually it should be able to roll back ok. But you never should unplug a disk with a mounted fs - and you can not umount it while it is in use. |
@real-yfprojects So one notification at the beginning of the backup, one at the end of the backup, one at the beginning of the consistency check, and one after the consistency check is complete? The smallest interval for the validation of the repository data is currently once a week. At that point it would be four notifications, instead of the usual two for a normal backup without validation. I wonder if this will be too many notifications and if this change would be an improvement.
@ThomasWaldmann What exactly does "roll back" mean in this context? |
@sudwhiwdh borg usually tries to work transaction based. So if you start from a valid repo state, it starts a transaction. Then it works on that and either successfully completes it (finishes transaction) or gets interrupted somehow (crash, power failure, ...). The next time it works with the repo, it will notice if there is an incomplete transaction and then roll back the repo files to that last valid state (== undo the not completed changes). |
We could reduce it to three notifications by dropping backup completed when there is a consistency check. In case of an additional prune that would make 4 notifications. Or we move the completed notification to the end of all three operations as you suggested. |
There is also a notification for pruning? |
No but we might want to add it because of your FR. |
@sudwhiwdh what would you prefer out of these two:
|
I can say what would be important to me as a user in relation to my use case. For me, it would be important to know when the backup starts and from when it is safe to unmount my external USB storage device again because the backup is completely finished. This includes any validations and pruning. Especially if there are several steps, I think it would be transparent to name them and be able to see them, but this can be looked up in the repository tab at any time. I would also find it overwhelming to receive a lot of notifications for a backup, because I am informed individually for each step that begins and ends. |
Requested Solution
Vorta displays the notification "Backup successful for [repository]" only after the data consistency check has been completed.
Alternatives
As an alternative, I could imagine Vorta sending out different notifications. First for the backup and separate notifications for the consistency check.
The text was updated successfully, but these errors were encountered: