----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7645/#review12562 -----------------------------------------------------------
docs/en-US/cannot-add-hosts.xml <https://reviews.apache.org/r/7645/#comment26818> This is factually inaccurate per edison docs/en-US/database-checklists.xml <https://reviews.apache.org/r/7645/#comment26819> I think you can have both FQDN or IP address - not sure that FQDN is mandatory. docs/en-US/database-checklists.xml <https://reviews.apache.org/r/7645/#comment26820> 6.2 is no longer a preferred platform and there are others that work as well - consider purging the entire bit docs/en-US/database-checklists.xml <https://reviews.apache.org/r/7645/#comment26821> CloudPlatform should be &PRODUCT; docs/en-US/mgmt-server-checklists.xml <https://reviews.apache.org/r/7645/#comment26822> 6.2 is no longer a preferred platform - there are also other platforms that work docs/en-US/mgmt-server-checklists.xml <https://reviews.apache.org/r/7645/#comment26823> A number of problems here: CloudPlatform sould be &PRODUCT; Also - the ISO is far less helpful as CloudStack now needs packages that aren't available on the first CD. docs/en-US/run-diagnostic-script.xml <https://reviews.apache.org/r/7645/#comment26824> Consider using <replaceable>link-local-ip</replaceable> docs/en-US/run-diagnostic-script.xml <https://reviews.apache.org/r/7645/#comment26825> Consider using replaceable here as well docs/en-US/storage-checklists.xml <https://reviews.apache.org/r/7645/#comment26826> There are far more options than this for storage - is there a reason we have limited it to these two? docs/en-US/storage-checklists.xml <https://reviews.apache.org/r/7645/#comment26828> NFS is no longer the only options for secondary storage docs/en-US/storage-checklists.xml <https://reviews.apache.org/r/7645/#comment26827> CloudStack should be &PRODUCT; docs/en-US/troubleshoot-install.xml <https://reviews.apache.org/r/7645/#comment26829> Why is this not a chapter? docs/en-US/troubleshooting-ssvm.xml <https://reviews.apache.org/r/7645/#comment26830> Second sentence doesn't seem to fit (actually it isn't a sentence at all, which is probably the problem) docs/en-US/troubleshooting-ssvm.xml <https://reviews.apache.org/r/7645/#comment26831> Second sentence seems misplaced - if this is a list of problems, is it a problem that it contacts dl.c.c? docs/en-US/vlan-issues.xml <https://reviews.apache.org/r/7645/#comment26832> I'd consider using 'hypervisor' instead of host. CloudStack uses the term host to refer to a number of things (see the return on the listHosts API command) - David Nalley On Oct. 18, 2012, 7:09 a.m., Radhika PC wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7645/ > ----------------------------------------------------------- > > (Updated Oct. 18, 2012, 7:09 a.m.) > > > Review request for cloudstack, David Nalley, Chip Childers, Jessica Tomechak, > and Wido den Hollander. > > > Description > ------- > > Troubleshooting section in the Install Guide > > > Diffs > ----- > > docs/en-US/binary-log-error.xml PRE-CREATION > docs/en-US/cannot-add-hosts.xml PRE-CREATION > docs/en-US/check-log-files.xml PRE-CREATION > docs/en-US/console-proxy-vm-issues.xml PRE-CREATION > docs/en-US/database-checklists.xml PRE-CREATION > docs/en-US/mgmt-server-checklists.xml PRE-CREATION > docs/en-US/preparation-checklists.xml PRE-CREATION > docs/en-US/run-diagnostic-script.xml PRE-CREATION > docs/en-US/storage-checklists.xml PRE-CREATION > docs/en-US/troubleshoot-awsapi.xml PRE-CREATION > docs/en-US/troubleshoot-install.xml PRE-CREATION > docs/en-US/troubleshoot-ms-log.xml PRE-CREATION > docs/en-US/troubleshooting-ssvm.xml PRE-CREATION > docs/en-US/vlan-issues.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/7645/diff/ > > > Testing > ------- > > patch cleanly applies > > > Thanks, > > Radhika PC > >
