-
-
Notifications
You must be signed in to change notification settings - Fork 58
* [JENKINS-45781] Remote ssh host management and configurable host na… #17
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
base: master
Are you sure you want to change the base?
Conversation
|
This PR would be great for upcoming release. It really helps if you connect to multiple ip based machines. |
|
IMHO this should be more in form of a "naming strategy" where you pick from combo-box whether you like the old, user@hostname or "aliased one" - it would be easier to implement custom ones e.g. based on job's binded credentials etc. |
|
But that's how it was prepared, isn't it? Name/Alias is optional and if it's empty it will fallback to default value user@hostname. |
|
I'm not against this PR, but IMHO if I'm defining an "alias" I expect the original information to be hidden (aliased) entirely - and in the PR it works more in a "prefix" way. |
|
I think, that this solution Name / alias - SSH host is better than just an Name / alias. And it does not matter if we call it alias or prefix. I think it will improve user experience. |
|
@ljader Otherwise, I have only one server, I wanna deploy a nodejs project though jenkins with docker on it, |
|
I think this is very useful, hope to be merged |
|
any news about this essential PR for environments where there are dozens of external servers? Thanks! :) |
|
I have created a release version with this PR https://github.com/Ykus/ssh-plugin/releases/tag/ssh-2.7-SNAPSHOT If the owner of this plugin will come up with exact definition how this improvements should work I can change the behavior. |
Adds possibility to name / alias SSH hosts.
Name / alias is displayed in select box (SSH site) where you can execute remote commands or scripts in project configuration.
@jenkinsci/code-reviewers