Hey Min, Nice job :-) Is there any change in licensing for the new SDK? We still have an issue with the distribution of the vmware stuff, I believe we also discussed moving to a more redistributable version of the vmware SDK (can't remember the name though), might it not be a good idea to put that in as well now?
As discussed in other threads as well, can you explain what you put into this feature branch with regards to automated testing? I'm particularly interested in the unit tests you put in to verify any changed parts of the code. If you want I can put a build job on this feature branch so we get the code coverage output and the unittest overview automagically on Jenkins. Cheers, Hugo > -----Original Message----- > From: Kelven Yang [mailto:kelven.y...@citrix.com] > Sent: donderdag 21 februari 2013 23:39 > To: cloudstack-dev@incubator.apache.org > Subject: Re: [DISCUSS]Upgrade Vmware SDK to 5.1 to support Windows 8 > and Windows 2012 as VM guest OS on vmware > > Please merge in > > Kelven > > On 2/21/13 2:18 PM, "Min Chen" <min.c...@citrix.com> wrote: > > >Hi there, > > > >Currently Cloudstack is not supporting Windows 8 and Windows Server > >2012 as VM guest OS on Vmware hypervisor because we are using Vmware > >4.1 web service SDK in our nonoss build. In Vmware SDK 4.1, Windows 8 > >and Windows > >2012 are not supported guest OS, and they are only fully supported for > >ESXi 5.1 and ESXi 5.0 Patch 4. To support Windows 8/2012 on vmware from > >Cloudstack, we need to upgrade our Vmware SDK dependency from 4.1 to > 5.1. > >There are several major changes that went into 5.1 SDK requiring some > >code changes in Cloudstack: > > > > * Vmware SDK 5.1 is generated using JAX-WS RI implementation bundled > >with Java 6 instead of previous Axis, so api signature is completely > >changed. Simply replacing vim25.jar will break compilation of > >CloudStack code. > > * Vmware SDK 5.1 for java client has gotten rid of apputils.jar that > >used to be in Vmware SDK 4.1, so we need to write our own wrapper layer > >to handle previous apputils methods. > > > >This feature (targeted for 4.2.0) is to fix CloudStack code to work > >with Vmware SDK 5.1 to be able to work with both Vcenter 5.1 and older > >version of Vcenter. Vcenter back-compatibilty support of the new 5.1 > >SDK is guaranteed by Vmware, see > >http://pubs.vmware.com/vsphere- > 51/index.jsp?topic=%2Fcom.vmware.wssdk.dsg. > >doc%2Fsdk_sg_introduction.3.6.html for details. > > > >JIRA ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-873 > > > >I have already had code working in my local feature branch vim51_win8, > >and done integration testing using Vcenter 5.1 and below, will merge to > >master after getting approval. > > > >Thanks > >-min > >
