You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Venkata Siva Vijayendra Bhamidipati <vi...@citrix.com> on 2013/03/04 22:21:11 UTC
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 an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8180/#review17361
-----------------------------------------------------------
Ship it!
Ship It!
- Venkata Siva Vijayendra Bhamidipati
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
>
>
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
Posted by Animesh Chaturvedi <an...@citrix.com>.
Murali
Is your relevant code in master now
Animesh
> -----Original Message-----
> From: Radhika PC [mailto:noreply@reviews.apache.org] On Behalf Of
> Radhika PC
> Sent: Tuesday, March 05, 2013 1:26 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 March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > > Ship It!
>
> will merge once Murali merges the code into Master.
>
>
> - Radhika
>
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8180/#review17361
> -----------------------------------------------------------
>
>
> 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
> >
> >
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
Posted by Radhika PC <ra...@citrix.com>.
> On March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > Ship It!
will merge once Murali merges the code into Master.
- Radhika
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8180/#review17361
-----------------------------------------------------------
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
>
>
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
Posted by Radhika PC <ra...@citrix.com>.
> On March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > Ship It!
>
> Radhika PC wrote:
> will merge once Murali merges the code into Master.
>
> Animesh Chaturvedi wrote:
> Radhika both 312 and 265 are closed now, can this review be closed now
submitted
commit b7f5197c850f772980b5f4f416878589c3d3e853
Author: unknown <radhikap@BANLRADHIKAP.citrite.net
Date: Wed Apr 24 23:06:25 2013 +0530
CLOUDSTACK-832, CLOUDSTACK-812,CLOUDSTACK-808
- Radhika
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8180/#review17361
-----------------------------------------------------------
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
>
>
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
Posted by Radhika Puthiyetath <ra...@citrix.com>.
Hi Animesh/ Murali
Thank you for the nudge.
Will submit once I am back to work.
-Radhika
-----Original Message-----
From: Animesh Chaturvedi [mailto:noreply@reviews.apache.org] On Behalf Of Animesh Chaturvedi
Sent: Wednesday, April 17, 2013 7:42 AM
To: Joe Brockmeier; Vijayendra Bhamidipati; Chip Childers; Jessica Tomechak; Murali Reddy
Cc: Radhika Puthiyetath; Animesh Chaturvedi; 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 March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > Ship It!
>
> Radhika PC wrote:
> will merge once Murali merges the code into Master.
Radhika both 312 and 265 are closed now, can this review be closed now
- Animesh
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8180/#review17361
-----------------------------------------------------------
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
>
>
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
Posted by Animesh Chaturvedi <An...@citrix.com>.
> On March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > Ship It!
>
> Radhika PC wrote:
> will merge once Murali merges the code into Master.
Radhika both 312 and 265 are closed now, can this review be closed now
- Animesh
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8180/#review17361
-----------------------------------------------------------
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
>
>