-
Notifications
You must be signed in to change notification settings - Fork 141
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
Give option to break repository lock #863
Conversation
This reverts commit c5a2e45. Turns out the tests break all the other tests. I don't know why
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.
maybe this should sound a bit more serious?
f"The repository at {repo_url} might be in use by another computer. Override it and continue?")) | ||
msg.accepted.connect(lambda: self.break_lock(profile)) |
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.
uuh, this almost sounds like recommending to break it.
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 thought the same actually. Here another take inspired by https://borgbackup.readthedocs.io/en/stable/usage/lock.html
The repository at {repo_url} might be in use elsewhere. Only proceed if you are certain no other Borg process on any machine is accessing the repository. Continue to break lock?
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.
Yeah, that is much better.
Could be also:
The repository at {repo_url} might be in use elsewhere. Only break the lock if you are certain no other Borg process on any machine is accessing the repository. Abort or break the lock?
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.
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.
Continue work on #718
Fixes #713, fixes #374