-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7661/#review12651
-----------------------------------------------------------

Ship it!


Applied to master.  I had to break appart the patch, due to a conflict.  
However, the content needed to be updated, so I made those changes by hand.  
There are two commits for this review:

commit 59091fde3370b50eb3763c6f418bb4a4874d9283
Author: Chip Childers <chip.child...@gmail.com>
Date:   Mon Oct 22 09:22:28 2012 -0400

    CLOUDSTACK-376: Changes to the min sys reqs doc.
    
    I had to manually apply the changes, due to a conflict when applying
    https://reviews.apache.org/r/7661.
    
    The rest of the patch submitted via that review request was committed
    in the previous master branch commit.
    
    Signed-off-by: Chip Childers <chip.child...@gmail.com>

commit 76478d963bbdfe48e4c83abc0cf8d3c97c499e2c
Author: Radhika PC <radhika.puthiyet...@citrix.com>
Date:   Fri Oct 19 13:29:03 2012 +0530

    partial-fix-for-376
    
    Signed-off-by: Radhika PC <radhika.puthiyet...@citrix.com>

- Chip Childers


On Oct. 19, 2012, 8:04 a.m., Radhika PC wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7661/
> -----------------------------------------------------------
> 
> (Updated Oct. 19, 2012, 8:04 a.m.)
> 
> 
> Review request for cloudstack, David Nalley, Jessica Tomechak, and edison su.
> 
> 
> Description
> -------
> 
> Fixed the following:
> 1. "Provde the CPU, memory, storage" in page 7, Provde should be "Provide"
> 2. "Also supported: CentOS/RHEL 5.5 or Ubuntu 10.04" in page 14, we don't 
> test 4.0 on this OS, there is no specific build for these OS also, so better 
> to remove them from the docs.(not present in the doc)
> 3. "• Statically allocated IP Address" can be removed, but with a notice, if 
> dhcp is used for hosts, make sure there is no conflict between dhcp server 
> used for these hosts and the dhcp router created by cloudstack.
> 4. "/usr/lib64/cloud/agent/scripts/storage/secondary/cloud-install-sys-tmplt 
> " in page 27: the path should be 
> "/usr/lib64/cloud/common/scripts/storage/secondary/cloud-install-sys-tmplt "
> 5. add download vhd-util in page 26
> 
> 
> This addresses bug CLOUDSTACK-376.
> 
> 
> Diffs
> -----
> 
>   docs/en-US/about-hosts.xml 956c695 
>   docs/en-US/management-server-install-client.xml 93273ce 
>   docs/en-US/management-server-install-multi-node.xml 8ac70ac 
>   docs/en-US/management-server-install-overview.xml 5bc6f1b 
>   docs/en-US/minimum-system-requirements.xml 7c0f15f 
>   docs/en-US/prepare-system-vm-template.xml 160cea4 
> 
> Diff: https://reviews.apache.org/r/7661/diff/
> 
> 
> Testing
> -------
> 
> Patch applies cleanly
> 
> 
> Thanks,
> 
> Radhika PC
> 
>

Reply via email to