Re: Review Request: Add docs for MidoNet networking plugin [CLOUDSTACK-996]

[Radhika PC <ra...@citrix.com>]

> On April 30, 2013, 8:01 a.m., Jessica Tomechak wrote:
> > docs/en-US/plugin-midonet-preparations.xml, line 40
> > <https://reviews.apache.org/r/10136/diff/4/?file=286089#file286089line40>
> >
> >     Believe the DTD requires a <para> tag within each entry tag. Like <entry><para>Setting Name</para></entry>. The same comment applies throughout the doc here.
> 
> Dave Cahill wrote:
>     Wouldn't Publican fail to build on DTD errors?
>     
>     Running this command successfully builds the doc for me:
>     publican build --config=publican-plugin-midonet.cfg --langs=en-US --format=pdf
> 
> Dave Cahill wrote:
>     The DocBook  site also shows this as an example of a legal entry (no para):
>     <entry>3.1415927</entry>
>     
>     ref: http://www.docbook.org/tdg/en/html/entry.html

Across the CloudStack doc, we use a block element, such as <para> element within a <entry> element. This is for using a list in a table entry, along with some text outside the list. 


- Radhika


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


On April 30, 2013, 5:22 a.m., Dave Cahill wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10136/
> -----------------------------------------------------------
> 
> (Updated April 30, 2013, 5:22 a.m.)
> 
> 
> Review request for cloudstack and Radhika PC.
> 
> 
> Description
> -------
> 
> Adding docs for MidoNet networking plugin [CLOUDSTACK-996]
> 
> Plugin itself is awaiting review / commit here:
> [PATCH 1] https://reviews.apache.org/r/9897/
> [PATCH 2] https://reviews.apache.org/r/9898/
> 
> 
> This addresses bug CLOUDSTACK-996.
> 
> 
> Diffs
> -----
> 
>   docs/en-US/MidoNet_Plugin_Guide.ent PRE-CREATION 
>   docs/en-US/MidoNet_Plugin_Guide.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-about.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-features.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-introduction.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-preparations.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-provider.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-revisions.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-ui.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-usage.xml PRE-CREATION 
>   docs/publican-plugin-midonet.cfg PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10136/diff/
> 
> 
> Testing
> -------
> 
> Built docs.
> 
> 
> Thanks,
> 
> Dave Cahill
> 
>