My mistake for interrupting in the thread.

--Alex

> -----Original Message-----
> From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
> Sent: Tuesday, January 15, 2013 5:24 PM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [IP Clearance: CLOUDSTACK-778] Feedback requested
> 
> So this really looks like two different issues. Vijay please create a separate
> [DISCUSS] thread for 715. Let's keep this thread only for community
> feedback on  778
> 
> > -----Original Message-----
> > From: Vijayendra Bhamidipati [mailto:vijayendra.bhamidip...@citrix.com]
> > Sent: Tuesday, January 15, 2013 5:13 PM
> > To: cloudstack-dev@incubator.apache.org
> > Subject: RE: [IP Clearance: CLOUDSTACK-778] Feedback requested
> >
> > Hi Alex!
> >
> > Actually the changes for this instance name feature were all in the
> > management layers and not in the resource layer and apply to all
> hypervisors,
> > so we would not be able to tie this change to overall vmware support
> > specifically in 4.1.
> >
> > Regarding 715, during the previous discussion that Kelven, yourself and I
> had,
> > Kelven had mentioned that we currently use the closed source client
> libraries
> > released by vmware, which as he pointed out in his comments in 715, have
> > changed a lot in vSphere 5.1 and infact break compile time compatibility
> with
> > cloudstack. The same compile time compatibility breakage was seen with
> the
> > open source 5.1 API also since it moves alongside the closed source version.
> >
> > To sum up the discussion we had, we would need to have both the closed
> > source vim25.jar file (for backward compatibility) and the newer vim25.jar
> > (that comes with vSphere 5.1) to work with newer vCenter features going
> > forward. This latter jar file we could either generate ourselves from the
> wsdl
> > files or possibly use the open source version of vi sdk's vim25.jar if the
> license
> > allows it. Either ways, we would need to modify existing cloudstack code
> for
> > vmware to separate it on the lines of how we do it for XAPI so that it uses
> the
> > right API calls depending on the version of vmware. This would be more
> > involved so it wouldn't be doable very quickly. Some cleanup with other jar
> files
> > like apputils.jar could also be required.
> >
> >
> > Regards,
> > Vijay
> >
> >
> > -----Original Message-----
> > From: Alex Huang [mailto:alex.hu...@citrix.com]
> > Sent: Tuesday, January 15, 2013 4:32 PM
> > To: cloudstack-dev@incubator.apache.org
> > Subject: RE: [IP Clearance: CLOUDSTACK-778] Feedback requested
> >
> > Hey Vijay,
> >
> > I just realized that this fix might mean we can finally make vmware support
> > part of oss.  Can you evaluate this against the following issue?
> >
> > https://issues.apache.org/jira/browse/CLOUDSTACK-715
> >
> > Is it something you can do very quickly for 4.1?  Kelven says there's still
> some
> > helper classes we have to replace to make it completely free of vmware
> > licensed code.
> >
> > Thanks.
> >
> > --Alex
> >
> > > -----Original Message-----
> > > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
> > > Sent: Tuesday, January 15, 2013 4:14 PM
> > > To: cloudstack-dev@incubator.apache.org
> > > Subject: RE: [IP Clearance: CLOUDSTACK-778] Feedback requested
> > >
> > > In that case it looks like a simple contribution and in my opinion may
> > > not need to go through IP clearance process. Folks if you have
> > > objection to this assessment please call out.
> > >
> > > Thanks
> > > Animesh
> > >
> > > > -----Original Message-----
> > > > From: Vijayendra Bhamidipati
> > > > [mailto:vijayendra.bhamidip...@citrix.com]
> > > > Sent: Tuesday, January 15, 2013 3:45 PM
> > > > To: cloudstack-dev@incubator.apache.org
> > > > Subject: RE: [IP Clearance: CLOUDSTACK-778] Feedback requested
> > > >
> > > > Hi Animesh,
> > > >
> > > > The feature has changes amounting to about 150 lines of code. It was
> > > intended
> > > > specifically for vCenter deployments that would allow users to
> > > > append
> > > guest
> > > > VM instance names to the internal names shown on the hypervisor
> > > > (i.e., in vCenter), if a global flag vm.instancename.flag is turned
> > > > on, during guest
> > > VM
> > > > creation. The feature however works on all hypervisors and has been
> > > coded
> > > > that way, but it doesn't impinge on any existing CS functionality
> > > > and should
> > > be
> > > > easy to merge into ACS. The only part that may need to be done
> > > > differently
> > > is in
> > > > the upgrade scripts from the previous ACS version to the current
> > > > one, and
> > > that
> > > > would only involve putting in a new global flag.
> > > >
> > > > Regards,
> > > > Vijay
> > > >
> > > >
> > > > -----Original Message-----
> > > > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
> > > > Sent: Tuesday, January 15, 2013 2:24 PM
> > > > To: cloudstack-dev@incubator.apache.org
> > > > Subject: [IP Clearance: CLOUDSTACK-778] Feedback requested
> > > >
> > > > Vijay
> > > >
> > > > This issue was flagged for potential IP clearance issue. Please
> > > > provide some more details on this issue and approximate effort in
> > > > LOC needed to address
> > > this
> > > > issue. Community needs to determine the contribution effort  and
> > > > then call
> > > out
> > > > whether it needs to go through IP clearance.
> > > >
> > > >
> > > > Thanks
> > > > Animesh
> > > >
> > > > > Chip Childers reopened CLOUDSTACK-778:
> > > > > --------------------------------------
> > > > >
> > > > >
> > > > > Reopening, since we (1) don't have the code in the repo and (2)
> > > > > need to sort out the ip ownership questions on this feature before
> > > > > inclusion.  Please feel free to submit to reviewboard, but note
> > > > > that we should NOT commit until ip clearance is complete.
> > > > >

Reply via email to