-
Notifications
You must be signed in to change notification settings - Fork 314
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
Support native cmd for ssh/scp #711
Comments
I think we could support that by adding a new argument to specify the path of an alternative SSH executable binary, something like |
Agree entirely. I'd want |
if tiup/pkg/cluster/executor/ssh.go Line 304 in 91c5f94
ssh := "ssh"
isExec := func(mode os.FileMode) bool { return mode&0111 == 0111 }
if env := os.Getenv(localdata.EnvNameNativeSSHPath); env != "" {
if info, err := os.Stat(env); err == nil && !info.IsDir() && isExec(info.Mode()) {
ssh = env
}
}
args := []string{ssh, "-o", "StrictHostKeyChecking"} |
@9547 Good advice. It's not a universal need, so ENV is OK. |
Feature Request
Is your feature request related to a problem? Please describe:
Nope.
Describe the feature you'd like:
tiup cluster executor can use native ssh. That's nice. But it can only use "ssh"(in PATH). We can't define the command we want to use.
tiup/pkg/cluster/executor/ssh.go
Line 304 in 91c5f94
Describe alternatives you've considered:
What if we use other ssh tools to connect the hosts? e.g. some ssh tools which contain security permissions.
Use
alias
could solve it. But it's not a good method.I think supporting native cmd is a good way.
Teachability, Documentation, Adoption, Migration Strategy:
The text was updated successfully, but these errors were encountered: