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

lib/fog/azurerm/models/compute/server.rb cannot list windows machines #36

Closed
mattohope opened this issue Jun 6, 2016 · 3 comments
Closed

Comments

@mattohope
Copy link

line 41 of lib/fog/azurerm/models/compute/server.rb assumes that the machine is a linux box and breaks if retrieving details of a windows host.

Current code:
hash['disable_password_authentication'] = vm['properties']['osProfile']['linuxConfiguration']['disablePasswordAuthentication']

Proposed "Quick Fix"
if vm['properties']['osProfile']['linuxConfiguration']
hash['disable_password_authentication'] = vm['properties']['osProfile']['linuxConfiguration']['disablePasswordAuthentication']
else
hash['disable_password_authentication'] = false
end

@abbas-sheikh-confiz
Copy link
Contributor

Hi matthope,

We are looking into this and will resolve it soon.

thanks for pointing out.

@shaffan-chaudhry-confiz
Copy link
Member

shaffan-chaudhry-confiz commented Jun 9, 2016

Hi @mattohope,

We have resolved this issue in this Pull Request: #54.
The gem is not updated on rubygems.org. You can install gem using GitHub.
Plz verify.

@mattohope
Copy link
Author

sorry i realized that i never came back to you about this, i have tested and this is no longer an issue

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

3 participants