cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jessica Tomechak" <jessica.tomec...@gmail.com>
Subject Re: Review Request: Add docs for MidoNet networking plugin [CLOUDSTACK-996]
Date Thu, 28 Mar 2013 02:24:26 GMT

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


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