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

fixes #19235 - vmware.rb : add Server 2016 support #4447

Closed
wants to merge 1 commit into from

Conversation

ricou84
Copy link

@ricou84 ricou84 commented Apr 10, 2017

Add support for Windows 2016 server image based provisionning

Add support for Windows 2016 server image based provisionning
@theforeman-bot
Copy link
Member

There were the following issues with the commit message:

  • 37fb6dd must be in the format fixes #redmine_number - brief description

If you don't have a ticket number, please create an issue in Redmine.

More guidelines are available in Coding Standards or on the Foreman wiki.


This message was auto-generated by Foreman's prprocessor

@theforeman-bot
Copy link
Member

Can an existing organization member please verify this patch?

2 similar comments
@theforeman-bot
Copy link
Member

Can an existing organization member please verify this patch?

@theforeman-bot
Copy link
Member

Can an existing organization member please verify this patch?

@mention-bot
Copy link

@ricou84, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ohadlevy, @timogoebel and @mmoll to be potential reviewers.

@ricou84 ricou84 changed the title vmware.rb : add Server 2016 support fixes #19235 - vmware.rb : add Server 2016 support Apr 10, 2017
@ricou84
Copy link
Author

ricou84 commented Apr 10, 2017

Seems not sufficient to support 2016

@mmoll
Copy link
Contributor

mmoll commented Apr 10, 2017

please see the URL in line 191, the whole dataset would need an update, if there's a new vSphere version available

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the vSphere 6.5 GuestOSIdentifier Documentation the name should be "windows9Server64Guest" instead of "windows9srv-64". If i understand it right, just the label "Microsoft Windows Server Threshhold (64-bit)" should be replaced with "Microsoft Windows Server 2016 (64-bit)"

http://pubs.vmware.com/vsphere-65/index.jsp?topic=%2Fcom.vmware.wssdk.apiref.doc%2Fvim.vm.GuestOsDescriptor.GuestOsIdentifier.html

Copy link
Contributor

@mmoll mmoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please either synchronize the whole list here, if that fixes the original issue or open another issue to synchronize the list and then continue to research the Win 2016 problem.

@ghost
Copy link

ghost commented May 19, 2017

@mmoll i revised the whole list from the new vSphere 6.5 documentation.
see #4543

@mmoll
Copy link
Contributor

mmoll commented Jul 5, 2017

GH-4617 got in, this can get closed.

@timogoebel timogoebel closed this Jul 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants