You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Jessica Tomechak <je...@gmail.com> on 2012/11/02 07:59:01 UTC

Re: Review Request: New Doc file for About password and key encryption. In other sections, fixed references to the same section. (defect cloudstack-403)

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


Re: Review Request: New Doc file for About password and key encryption. In other sections, fixed references to the same section. (defect cloudstack-403)

Posted by Radhika PC <ra...@citrix.com>.

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