----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10136/#review18453 -----------------------------------------------------------
docs/en-US/plugin-midonet-about.xml <https://reviews.apache.org/r/10136/#comment38680> Nice job chunking the doc into sections with separate XML files. docs/en-US/plugin-midonet-features.xml <https://reviews.apache.org/r/10136/#comment38676> Suggest making this note on KVM support the first thing readers see. Place at the top of plugin-midonet-introduction.xml. docs/en-US/plugin-midonet-introduction.xml <https://reviews.apache.org/r/10136/#comment38675> Don't think you meant to use the footnote tag here. docs/en-US/plugin-midonet-introduction.xml <https://reviews.apache.org/r/10136/#comment38677> You used &PRODUCT; instead of CloudStack in some places but not all. There are several instances in several files. Suggest using &PRODUCT; consistently. docs/en-US/plugin-midonet-preparations.xml <https://reviews.apache.org/r/10136/#comment38678> Suggest presenting this as an <orderedlist> rather than paragraphs. docs/en-US/plugin-midonet-revisions.xml <https://reviews.apache.org/r/10136/#comment38681> We don't always use the Revision History in the CS docs, but I don't think we have any rule against it either. Your call, and maybe worth discussing on the mailing list. docs/en-US/plugin-midonet-ui.xml <https://reviews.apache.org/r/10136/#comment38679> Can you provide the exact UI steps to navigate to a physical network? i.e. "1. In the left navbar, click Infrastructure. 2. In Zones, click View All. 3. Click the name of the zone you want. 4. Click the Physical Network tab. 5. Click the name of the network you want." - Jessica Tomechak On March 27, 2013, 1:34 a.m., Dave Cahill wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10136/ > ----------------------------------------------------------- > > (Updated March 27, 2013, 1:34 a.m.) > > > Review request for cloudstack, Hugo Trippaers and Chiradeep Vittal. > > > 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 > >
