----------------------------------------------------------- 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 > >