On Thu, Apr 3, 2014 at 3:59 AM, Konstantina Chremmou <konstantina.chrem...@citrix.com> wrote: >> -----Original Message----- >> From: Alex Huang [mailto:alex.hu...@citrix.com] >> Sent: 02 April 2014 10:46 PM >> To: dev@cloudstack.apache.org >> Subject: [ANNOUNCE] Better XenServer support in 4.4.... >> >> I've talked about this all the way back when we were in Amsterdam and now >> it's finally done. Tina (Konstantina Chremmou) checked in a patch that >> removes CloudStack's own copy of XenServerJava source code and >> submitted a copy of the xen-api.jar into the maven repository. Since xen- >> api.jar is backwards compatible with previous versions of XenServer, only >> one copy of such jar is needed. >> >> For those of you not familiar with this, CloudStack keeps its own copy of >> three files that really belongs to XenServer: >> - xen-api.jar: CloudStack modified the source code to add a client >> side timeout to fault isolate CloudStack from XenServer if the XenServer >> control layer runs into trouble. >> - vhd-util: The copy of vhd-util shipped with XenServer is old and >> does not provide the functionality to change the parent id of the vhd file. >> - NFSSR.py: XenServer's copy always creates a subdirectory and >> utilize that subdirectory for its vm images. CloudStack needed one that >> doesn't create a subdirectory. >> >> With the release of hot fix XS62ESP1004, XenSever has incorporated all of >> CloudStack's changes for the three files. Unfortunately, these changes are >> not back-ported to previous versions so CloudStack will only utilize the new >> changes against XenSever 6.2 + SP1 + XS62ESP1004. There is a new resource, >> XenServer625Resource.java, that was added in 4.3 to work with this exact >> XenServer patch level. Unfortunately, the xen-api.jar couldn't make it in >> time for the 4.3 release so we still had to keep our own copy of the source >> code in 4.3. > > > We could still change it for 4.3-forward though. I've submitted for > consideration a patch for that branch too. > >
Doesn't sound like a bug fix, so probably not appropriate for 4.3-forward. As an aside, we need to make sure LICENSE gets updated.