> On Nov. 2, 2012, 6:59 a.m., Jessica Tomechak wrote:
> > 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.

will add a diff patch.


> On Nov. 2, 2012, 6:59 a.m., Jessica Tomechak wrote:
> > docs/en-US/about-password-encryption.xml, line 48
> > <https://reviews.apache.org/r/7804/diff/1/?file=183410#file183410line48>
> >
> >     Change all instances of CloudPlatform into &PRODUCT;

fixed


> On Nov. 2, 2012, 6:59 a.m., Jessica Tomechak wrote:
> > docs/en-US/management-server-install-db-local.xml, line 94
> > <https://reviews.apache.org/r/7804/diff/1/?file=183412#file183412line94>
> >
> >     Left out the period at the end of the sentence, like 
> > -encryption"/>.</para>

fixed


> On Nov. 2, 2012, 6:59 a.m., Jessica Tomechak wrote:
> > docs/en-US/management-server-install-db-local.xml, line 100
> > <https://reviews.apache.org/r/7804/diff/1/?file=183412#file183412line100>
> >
> >     Left out the period at the end of the sentence, like 
> > -encryption"/>.</para>

fixed


- Radhika


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


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

Reply via email to