-
Notifications
You must be signed in to change notification settings - Fork 155
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
Long running pre-backup commands freeze UI #210
Comments
Interesting, since the pre/post-backup command runs in its own thread. Will test. |
I just tested the pre-/post-backup commands in Vorta 0.6.10 and only the pre-backup command seems to freeze the UI. I assumed it extends to the post command as well, but it does not. |
That makes sense actually. The pre-backup command is run when preparing the backup. So it runs in the main thread. (I gave the wrong info just before.) The reason for this is to allow checking for certain conditions. E.g. the right network. If the returncode is not 0, it will cancel the backup. I also noticed that the UI only blocks if the backup is started by the user. It doesn't block when the backup is started by the background scheduler. So I see those options to improve the behavior:
|
I wouldn't limit pre-backup to "checking conditions". Often this is used to bring the data into a backup-worthy state (like dumping an active database, doing a snapshot, etc.) and this obviously must always run, no matter whether started manually or automatically. So option 1 is out of question. 3 sounds like the best option. |
3 it is then. |
They are also uncancellable, while blocking other actions such as repo checking. |
I would try to have an own PreBackupJob inheriting JobInterface or BorgJob and scheduling the BorgCreateJob in its |
I have a pre backup script which takes a few minutes. It took me a while to realise that the UI had froze, without any notification that it was running. I know in the long term it would be good to implement option 3. Would it be possible, for the short term, to implement option 2 as this would allow me (and other users) to see that the backup (and pre backup script) is running, while the UI is frozen. |
Hi, |
Describe the bug
I was trying to add a random delay of 0-59 minutes per backup using the pre-backup command option. To do so I used the following command:
sleep $[ $RANDOM % 60 ]m
, but unfortunately while pre-backup commands are running, the Vorta UI freezes. This probably won't be an issue for most scripts, but will be a problem in cases where scripts take a long time to compete.To Reproduce
Steps to reproduce the behavior:
sleep 1m
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: