On Wed, Dec 5, 2012 at 12:38 AM, Devdeep Singh <devdeep.si...@citrix.com> wrote: > >> -----Original Message----- >> From: David Nalley [mailto:da...@gnsa.us] >> Sent: Tuesday, December 04, 2012 9:44 PM >> To: cloudstack-dev@incubator.apache.org >> Subject: Re: Adding support for XenServer 6.1 and Storage XenMotion >> >> > 2. Update the java bindings that are used by CloudStack for making xapi >> > calls. >> The current bindings are present under deps/XenServerJava in source tree. >> They need to be updated for getting the new apis that have been added for >> supporting storage xenmotion. CloudStack has made customizations to these >> bindings to address certain issues. They'll also be ported. >> >> Hi Devdeep: >> >> Thanks for starting this discussion early. >> In your plans to update the java bindings for XS, are you talking a forklift >> (e.g. >> remove the existing bindings, replace it with a new version and then patch >> the >> new version?) >> >> Any chance we could get the customizations upstreamed and then merely >> rely on the package as a dependency rather than actually carrying the source >> and building it each time? >> >> --David > > Hi David, > > Yes, my plan was to replace the bindings with the new version and then to > patch > the new version. > > We had asked the Xenserver team to upstream the customizations. I'll follow up > with them and find out what their plans are around it. > > Regards, > Devdeep >
Hi Devdeep! I'm looking at the set of 3 review requests you sent in, and wondering if you have any update on upstreaming the CS patch? I'll look at the reviews themselves, but I don't want us to lose sight of seeing if we can avoid having these source files in our repo! -chip
