----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7804/#review13028 -----------------------------------------------------------
Need an <xi:include> statement to actually include the new about-password-encryption.xml somewhere in the Installation Guide. Maybe inside management-server-install-flow.xml, right after management-server-install-db.xml. docs/en-US/about-password-encryption.xml <https://reviews.apache.org/r/7804/#comment28039> Change all instances of CloudPlatform into &PRODUCT; docs/en-US/management-server-install-db-local.xml <https://reviews.apache.org/r/7804/#comment28041> Left out the period at the end of the sentence, like -encryption"/>.</para> docs/en-US/management-server-install-db-local.xml <https://reviews.apache.org/r/7804/#comment28042> Left out the period at the end of the sentence, like -encryption"/>.</para> - Jessica Tomechak On Oct. 31, 2012, 11:20 a.m., Radhika PC wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7804/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2012, 11:20 a.m.) > > > Review request for cloudstack, David Nalley, Chip Childers, Jessica Tomechak, > and Wido den Hollander. > > > Description > ------- > > New Doc file on About password and key encryption. In the Install guide, > fixed references to the same section. > > > This addresses bug cloudstack-403. > > > Diffs > ----- > > docs/en-US/about-password-encryption.xml PRE-CREATION > docs/en-US/management-server-install-db-external.xml a749dc7 > docs/en-US/management-server-install-db-local.xml 9880c54 > docs/en-US/prepare-system-vm-template.xml 5ed78f7 > > Diff: https://reviews.apache.org/r/7804/diff/ > > > Testing > ------- > > tested whether patch applies. worked fine > > > Thanks, > > Radhika PC > >