incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vijayendra Bhamidipati <vijayendra.bhamidip...@citrix.com>
Subject RE: [IP Clearance: CLOUDSTACK-778] Feedback requested
Date Thu, 17 Jan 2013 00:29:26 GMT
Please find the patch/signature/md5sum files at the following github repo:

https://github.com/vijayendrabvs/instanceNameCS.git

The files are in a zipped file (instanceNamediff.zip) in the repo.

Regards,
Vijay


-----Original Message-----
From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com] 
Sent: Tuesday, January 15, 2013 5:47 PM
To: cloudstack-dev@incubator.apache.org
Subject: RE: [IP Clearance: CLOUDSTACK-778] Feedback requested

Vijay

Please post the patch for community review and post the link in response to this thread

Animesh

> -----Original Message-----
> From: Chip Childers [mailto:chip.childers@sungard.com]
> Sent: Tuesday, January 15, 2013 5:30 PM
> To: <cloudstack-dev@incubator.apache.org>
> Subject: Re: [IP Clearance: CLOUDSTACK-778] Feedback requested
> 
> Can we see the patch in question?
> 
> - chip
> 
> On Jan 15, 2013, at 7:14 PM, Animesh Chaturvedi 
> <animesh.chaturvedi@citrix.com> wrote:
> 
> > 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.bhamidipati@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.chaturvedi@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.
> >

Mime
View raw message