On Thu, Apr 3, 2014 at 1:28 PM, David Nalley <da...@gnsa.us> wrote:
> 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.


I have a patch for this, and will push as soon as the LDAP issues are resolved.

--David

Reply via email to