Skip to content
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

Merged
merged 43 commits into from
Feb 17, 2021
Merged

Conversation

m3nu
Copy link
Contributor

@m3nu m3nu commented Feb 16, 2021

Continue work on #718

Fixes #713, fixes #374

@m3nu m3nu merged commit 848bcc5 into borgbase:master Feb 17, 2021
@m3nu m3nu deleted the pr/locked_repo branch February 17, 2021 01:58
Copy link
Collaborator

@ThomasWaldmann ThomasWaldmann left a 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?

Comment on lines +189 to +190
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))
Copy link
Collaborator

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.

Copy link
Contributor Author

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?

Copy link
Collaborator

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?

Copy link
Contributor Author

@m3nu m3nu Feb 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That will work too. I played a bit with the buttons, icon and text to make it more scary.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Notify user of locked repo Do manual break-lock from UI
3 participants