Hi David, Please mark it as submitted. I have applied your patch on Master and 4.1
-Radhika From: Sebastien Goasguen [mailto:nore...@reviews.apache.org] On Behalf Of Sebastien Goasguen Sent: Saturday, March 23, 2013 2:35 PM To: Radhika Puthiyetath; Jessica Tomechak Cc: Sebastien Goasguen; David Grizzanti; cloudstack Subject: Re: Review Request: CLOUDSTACK-1714/CLOUDSTACK-1715: Updates to Install and Admin Guide documentation This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10045/ On March 23rd, 2013, 12:53 a.m., Jessica Tomechak wrote: Ship It! Jessica, when you ship it, can you also apply it. thanks - Sebastien On March 22nd, 2013, 4:38 a.m., David Grizzanti wrote: Review request for cloudstack, Radhika PC and Jessica Tomechak. By David Grizzanti. Updated March 22, 2013, 4:38 a.m. Description CLOUDSTACK-1714: Doc section has wrong title: Setting Zone VLAN and Running VM Maximums CLOUDSTACK-1715: Missing "host" config setting in docs on management server load balancing Updates to Install & Admin Guide documentation: - Fixed section title for Management Server Load Balancing - Added description of updating 'host' global config value when setting up management server load balancer - Added reference to Management Server Load Balancing in Admin Guide with HA setup Bugs: CLOUDSTACK-1714/CLOUDSTACK-1715 Diffs * docs/en-US/management-server-install-multi-node.xml (3f011b8) * docs/en-US/management-server-lb.xml (85a8622) * docs/en-US/sys-reliability-and-ha.xml (94385ff) View Diff<https://reviews.apache.org/r/10045/diff/>
