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

Documentation of winrm options #105

Open
martinmosegaard opened this issue Mar 23, 2017 · 2 comments
Open

Documentation of winrm options #105

martinmosegaard opened this issue Mar 23, 2017 · 2 comments

Comments

@martinmosegaard
Copy link

From the README, it looks like the winrm options should be set under customization_spec.
But from driver.rb in the source code, it seems to be read from [:bootstrap_options][:ssh].

Maybe the bootstrap ssh options could be documented together in their own section?

@michaeltlombardi
Copy link

While not very intuitive, the README does say that the ssh options also apply to WinRM:

  • [:ssh][:user] user to use for ssh/winrm (defaults to root on linux/administrator on windows)
  • [:ssh][:password] - password to use for ssh/winrm
  • [:ssh][:paranoid] - specifies the strictness of the host key verification checking
  • [:ssh][:port] port to use for ssh/winrm (defaults to 22 for ssh or 5985 for winrm)

Whereas the machine options in customization_spec are for options not covered in the above - things like SSL, timeout, etc.

@martinmosegaard
Copy link
Author

I may be wrong, but if we look at the code here:
https://github.com/CenturyLinkCloud/chef-provisioning-vsphere/blob/master/lib/chef/provisioning/vsphere_driver/driver.rb

  • Then, in line 660, create_winrm_transport, options includes winrm_transport and winrm_opts.
  • create_winrm_transport is called from transport_for, above.
  • transport_for is called with ssh options, not customization_spec, as in:
transport_for(
        machine_spec,
        machine_options[:bootstrap_options][:ssh]

I did some brief testing and it seemed I had to put the winrm stuff under ssh, not customization_spec.

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

No branches or pull requests

2 participants