You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Radhika Puthiyetath <ra...@citrix.com> on 2013/02/21 03:36:15 UTC
RE: ptional Public IP assignment for EIP with Basic Zone, Enable
L4-L7 network services in the shared network in the advanced zone, and
User-provided hostname in vCenter
Hi Murali/Vijay
I have the doc patch at https://reviews.apache.org/r/8180/ left abandoned on the review board for long.
Let me know whether the following features merged to Master:
CLOUDSTACK-265, optional public IP for EIP with basic zone
CLOUDSTACK-312, L4-L7 network services
CLOUDSTACK-778, user-provided hostname
The review comment states that they are not yet merged to master. Please direct.
If they are merged, I shall resubmit the patch, because a file in this patch is already updated for Persistent network.
Thanks
-Radhika
-----Original Message-----
From: Radhika Puthiyetath [mailto:radhika.puthiyetath@citrix.com]
Sent: Thursday, February 21, 2013 7:33 AM
To: Animesh Chaturvedi; cloudstack-dev@incubator.apache.org; Chip Childers; Joe Brockmeier; Vijayendra Bhamidipati; Murali Reddy; Jessica Tomechak
Subject: RE: Review Request: Documentation for the new features: Optional Public IP assignment for EIP with Basic Zone, Enable L4-L7 network services in the shared network in the advanced zone, and User-provided hostname in vCenter
This is very much relevant! Nobody pushed ii for some reasons, and now some of the files in the patch is overwritten, and I need to submit a new patch based on latest files.
I am currently busy with 4.1 features. Once I am through with this , I shall update the diff .
-----Original Message-----
From: Animesh Chaturvedi
Sent: Thursday, February 21, 2013 5:02 AM
To: cloudstack-dev@incubator.apache.org; Radhika Puthiyetath; Chip Childers; Joe Brockmeier; Vijayendra Bhamidipati; Murali Reddy; Jessica Tomechak
Subject: RE: Review Request: Documentation for the new features: Optional Public IP assignment for EIP with Basic Zone, Enable L4-L7 network services in the shared network in the advanced zone, and User-provided hostname in vCenter
Radhika
Is this patch still relevant? If not please discard and submit a new patch
> -----Original Message-----
> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> Sent: Friday, February 15, 2013 9:54 AM
> To: cloudstack-dev@incubator.apache.org; Radhika Puthiyetath; Chip
> Childers; Joe Brockmeier; Vijayendra Bhamidipati; Murali Reddy;
> Jessica Tomechak
> Subject: RE: Review Request: Documentation for the new features:
> Optional Public IP assignment for EIP with Basic Zone, Enable L4-L7
> network services in the shared network in the advanced zone, and
> User-provided hostname in vCenter
>
> > -----Original Message-----
> > From: Radhika PC [mailto:noreply@reviews.apache.org] On Behalf Of
> > Radhika PC
> > Sent: Thursday, February 14, 2013 7:42 AM
> > To: Chip Childers; Joe Brockmeier; Vijayendra Bhamidipati; Murali
> > Reddy; Jessica Tomechak
> > Cc: Radhika Puthiyetath; cloudstack
> > Subject: Re: Review Request: Documentation for the new features:
> > Optional Public IP assignment for EIP with Basic Zone, Enable L4-L7
> > network services in the shared network in the advanced zone, and
> > User-provided hostname in vCenter
> >
> >
> >
> > > On Feb. 6, 2013, 3:21 a.m., Jessica Tomechak wrote:
> > > > CLOUDSTACK-265, optional public IP for EIP with basic zone:
> > > > passed IP
> > clearance 1/30, but comparing the proposed code at
> > http://people.apache.org/~muralireddy/ to the current code, I do not
> > see it in
> > 4.1 or master branches. CLOUDSTACK-312, L4-L7 network services:
> > passed IP clearance 1/30, but comparing the proposed code at
> > http://people.apache.org/~muralireddy to the repo, I do not see it
> > in
> > 4.1 or master branches. CLOUDSTACK-778, user-provided hostname,
> > isn't in 4.1. See Vijay B's comment in the bug today: "The patch
> > didn't make it to 4.1.0. Moving to 4.2.0." I have emailed Murali to
> > ask the status of these
> features.
> > >
> > > Radhika PC wrote:
> > > Once i get the confirmation from Murali, i will resubmit the
> > > patch
> > >
> > > Radhika PC wrote:
> > > This patch is applicable only for 4.2 as patch for both bugs
> > > 312, 265 did not
> > go in to 4.1. Confirmed with Murali.
> >
> > since creating-network-offering.xml file has been changed now with
> > persistent network patch, I shall submit a new patch for this.
> >
> >
> [Animesh>] Radhika so is this patch not relevant anymore? If so please
> discard it
> > - Radhika
> >
> >
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/8180/#review16162
> > -----------------------------------------------------------
> >
> >
> > On Jan. 15, 2013, 12:34 p.m., Radhika PC wrote:
> > >
> > > -----------------------------------------------------------
> > > This is an automatically generated e-mail. To reply, visit:
> > > https://reviews.apache.org/r/8180/
> > > -----------------------------------------------------------
> > >
> > > (Updated Jan. 15, 2013, 12:34 p.m.)
> > >
> > >
> > > Review request for cloudstack, Chip Childers, Venkata Siva
> > > Vijayendra
> > Bhamidipati, Jessica Tomechak, Murali Reddy, and Joe Brockmeier.
> > >
> > >
> > > Description
> > > -------
> > >
> > > This review request is for the documentation written for the
> > > following
> > features:
> > >
> > > Optional Public IP assignment for EIP with Basic Zone
> > > User-provided host name in vCenter Enable L4-L7 network services
> > > in the shared network in the advanced zone
> > >
> > >
> > > Diffs
> > > -----
> > >
> > > docs/en-US/Installation_Guide.xml 2f60aca
> > > docs/en-US/about-working-with-vms.xml 47153e2
> > > docs/en-US/append-displayname-vms.xml PRE-CREATION
> > > docs/en-US/creating-network-offerings.xml ab56920
> > > docs/en-US/elastic-ip.xml PRE-CREATION
> > > docs/en-US/network-offerings.xml c1fd79d
> > > docs/en-US/networks.xml a7b9ea1
> > > docs/en-US/set-up-network-for-users.xml 2b40923
> > > docs/en-US/using-multiple-guest-networks.xml 9076a81
> > > docs/en-US/using-netscaler-load-balancers.xml c2044de
> > > docs/en-US/virtual-machines.xml 7c74932
> > >
> > > Diff: https://reviews.apache.org/r/8180/diff/
> > >
> > >
> > > Testing
> > > -------
> > >
> > > Patch cleanly applies. doc is reviewed by QA
> > >
> > >
> > > Thanks,
> > >
> > > Radhika PC
> > >
> > >