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

set default borg_ssh_command #133

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kenayagi
Copy link
Contributor

Without a default borg_ssh_command, borg_ssh_key_* seem useless.

@m3nu
Copy link
Collaborator

m3nu commented Jun 14, 2023

Not useless, since Borgmatic will use its own default. (which is fine in most cases)

You can still override the option, if you need to pass custom options.

@kenayagi
Copy link
Contributor Author

I'm not that sure, but I could be missing something.

For example: if you only specify a custom borg_ssh_key_name, the role will use it to generate a new pair, but not for authenticating to the remote server with that public key.

@m3nu
Copy link
Collaborator

m3nu commented Jun 14, 2023

That's right. If you make your own key you also need to adjust this command. We had it like this during the last big refactor.

The issue with this is that you clutter the config file for the majority of users using standard keys and need to maintain this command.

@kenayagi
Copy link
Contributor Author

I do not agree: if I change the ssh key I'll have to remember to also change the ssh command. (You can read: human error incoming 😅).
With this simple default, instead, the role will edit it accordingly. And if a user need another custom command he can overwrite it.
However, please feel free to close the PR. I've only thought it could have been useful.

@m3nu
Copy link
Collaborator

m3nu commented Jun 15, 2023

Both options aren't great. I already thought about it when someone else suggested it in the big refactoring PR.

  • Set both the key name and the command when using a custom key
  • Hard-code the SSH command and add it in the Borgmatic config for everyone

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.

2 participants