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.

Sign up for GitHub

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

Jump to bottom

WinRM :port setting fails validation #25

Closed
MattVonVielen opened this issue Sep 22, 2015 · 1 comment
Closed

WinRM :port setting fails validation #25

MattVonVielen opened this issue Sep 22, 2015 · 1 comment

Comments

@MattVonVielen
Copy link

Tried using the example for WinRM in the README (edited for correct IP): https://github.com/chef/chef-provisioning-ssh/blame/master/README.md#L110-L118

My chef-client run errored with:

    RuntimeError
    ------------
    Machine Options for builder-windows-local are invalid cannot create machine.

    Resource Declaration:
    ---------------------

After debugging, I found the culprit is here: https://github.com/chef/chef-provisioning-ssh/blob/master/lib/chef/provisioning/ssh_driver/driver.rb#L222-L223

The unless extra == :port causes the validation error message to be suppressed, but the validation still fails (the valid = false on the next line).
@christinedraper
Copy link
Contributor

Problem still exists, reported on forum https://discourse.chef.io/t/bootstraping-windows-node-using-machine-resource/8490

christinedraper pushed a commit to christinedraper/chef-provisioning-ssh that referenced this issue May 18, 2016
Add option to store machine data in node attribute rather than file.
3fa00e0 
To use this option, specify a driver url of 'ssh:chef'

In support of above:
1. Some refactoring of driver.rb.
2. Fixes some warnings with gemspec.
3. Fixes tests for changed location of vagrant insecure key in vagrant 1.7+.
4. Updated readme for above and some out of date info.

Also fixes chef-boneyard#29 by using stored transport options if none specified, and
fixes chef-boneyard#25 so validation logic accepts windows :port
@christinedraper christinedraper mentioned this issue May 18, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MattVonVielen @christinedraper