-
Notifications
You must be signed in to change notification settings - Fork 27
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
JcloudsLocation for port-forwarding #1082
JcloudsLocation for port-forwarding #1082
Conversation
- Refactors methods for creation of users - Adds JcloudsPortForwarderExtension - Calls out to port-forwarder for port 22, if configured
Brooklyn Central » brooklyn #1379 SUCCESS |
sshProps.put("port", port); | ||
if (initialPassword.isPresent()) sshProps.put("password", initialPassword.get()); | ||
if (initialPrivateKey.isPresent()) sshProps.put("privateKeyData", initialPrivateKey.get()); | ||
if (initialPrivateKey.isPresent()) sshProps.put("privateKeyData", initialPrivateKey.get()); |
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.
Why do this twice?
@aledsage I've made minor comments that I think are worth addressing, but otherwise this looks good. New live test passed on AWS. As @richardcloudsoft mentioned, is there anything to go alongside this for the brooklyn-networking project? |
The changes in this pull request cause the build of the cloudstack module in brooklyn-networking to fail:
|
@sjcorbett @richardcloudsoft thanks, I've incorporated those comments (see the most recent commit). Are you happy for me to merge now? I've updated the brooklyn-networking downstream project at https://github.com/cloudsoft/brooklyn-networking/pull/3 |
@aledsage Go for it. |
…rtforwarding JcloudsLocation for port-forwarding
- in brooklyncentral#1082 a superset of these changes were done in master
Some changes I've had sitting around for far too long, that I've finally revisited! These were originally started in github.com/cloudsoft/brooklyn-networking