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:[email protected]] > Sent: Tuesday, January 15, 2013 5:13 PM > To: [email protected] > 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:[email protected]] > Sent: Tuesday, January 15, 2013 4:32 PM > To: [email protected] > 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:[email protected]] > > Sent: Tuesday, January 15, 2013 4:14 PM > > To: [email protected] > > 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:[email protected]] > > > Sent: Tuesday, January 15, 2013 3:45 PM > > > To: [email protected] > > > 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:[email protected]] > > > Sent: Tuesday, January 15, 2013 2:24 PM > > > To: [email protected] > > > 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. > > > >
