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

WinRM transport and port options #8

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

Conversation

monkey1016
Copy link

Adding a few more options to the transport_options, specifically transport and port.
Also determining the scheme based off of the transport type.

@monkey1016
Copy link
Author

Build failed because of gem dependency resolution. Does anyone have pointers by chance on how to resolve this?

@jjasghar
Copy link

@monkey1016 I'll update the .travis.yml and you'll need to rebase of that.

I'll post here when i get it done.

@jjasghar
Copy link

#9 when this is merged, please rebase off master.

@jjasghar
Copy link

Please rebase off master, and lets see the tests pass 🤘

@monkey1016
Copy link
Author

Thanks! Looks like the travis build is getting further now. I'll take it from here.

@monkey1016 monkey1016 closed this Sep 13, 2016
@monkey1016 monkey1016 reopened this Sep 13, 2016
@monkey1016 monkey1016 closed this Sep 13, 2016
@monkey1016 monkey1016 reopened this Sep 13, 2016
@monkey1016
Copy link
Author

Ok. I think I fixed all the warnings and issues now.

@jjasghar
Copy link

Awesome, thanks for that. I'm still trying to get vRA to test this; when I do and verify I'll merge. Thanks for the PR! 🤘

@monkey1016
Copy link
Author

No problem. Glad if it can help.

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

Successfully merging this pull request may close these issues.

2 participants