cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dave Cahill" <dcah...@midokura.com>
Subject Re: Review Request: Add docs for MidoNet networking plugin [CLOUDSTACK-996]
Date Tue, 30 Apr 2013 08:15:11 GMT


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

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


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message