incubator-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: doc files for inter vlan routing
Date Thu, 27 Sep 2012 06:49:46 GMT

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


Some of the image file names don't match the introductory text, which makes me wonder whether
it's the correct image file. It might be, if the same file is being reused for different tasks.
Minor issue: some of the alt texts don't match the image file names.


docs/en-US/delete-reset-vpn.xml
<https://reviews.apache.org/r/7106/#comment25519>

    This alt text doesn't match the .png filename.



docs/en-US/delete-reset-vpn.xml
<https://reviews.apache.org/r/7106/#comment25520>

    This alt text doesn't match the .png file name.



docs/en-US/release-ip-for-vpc.xml
<https://reviews.apache.org/r/7106/#comment25521>

    Shouldn't this refer to release-ip-icon.png?



docs/en-US/release-ip-for-vpc.xml
<https://reviews.apache.org/r/7106/#comment25522>

    This alt text has a typo, an extra "e" at the end. Also doesn't exactly match the .png
file name.



docs/en-US/remove-vpc.xml
<https://reviews.apache.org/r/7106/#comment25523>

    This is correct only if the icon for removing a VPC is the same as the icon for destroying
an instance.



docs/en-US/remove-vpc.xml
<https://reviews.apache.org/r/7106/#comment25524>

    This alt text doesn't match the current .png file name in the imagedata tag.



docs/en-US/remove-vpc.xml
<https://reviews.apache.org/r/7106/#comment25526>

    This alt text doesn't match the .png file name in the imagedata tag.



docs/en-US/remove-vpc.xml
<https://reviews.apache.org/r/7106/#comment25525>

    This alt text doesn't match the .png file name in the imagedata tag.


- Jessica Tomechak


On Sept. 14, 2012, 12:36 p.m., Radhika PC wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7106/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2012, 12:36 p.m.)
> 
> 
> Review request for cloudstack and David Nalley.
> 
> 
> Description
> -------
> 
> files for VPC/inter vlan routing
> 
> 
> Diffs
> -----
> 
>   docs/en-US/acquire-new-ip-for-vpc.xml PRE-CREATION 
>   docs/en-US/add-gateway-vpc.xml PRE-CREATION 
>   docs/en-US/add-loadbalancer-rule-vpc.xml PRE-CREATION 
>   docs/en-US/add-portforward-rule-vpc.xml PRE-CREATION 
>   docs/en-US/add-tier.xml PRE-CREATION 
>   docs/en-US/add-vm-to-tier.xml PRE-CREATION 
>   docs/en-US/add-vpc.xml PRE-CREATION 
>   docs/en-US/configure-acl.xml PRE-CREATION 
>   docs/en-US/configure-vpc.xml PRE-CREATION 
>   docs/en-US/delete-reset-vpn.xml PRE-CREATION 
>   docs/en-US/enable-disable-static-nat-vpc.xml PRE-CREATION 
>   docs/en-US/inter-vlan-routing.xml PRE-CREATION 
>   docs/en-US/release-ip-for-vpc.xml PRE-CREATION 
>   docs/en-US/remove-tier.xml PRE-CREATION 
>   docs/en-US/remove-vpc.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/7106/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Radhika PC
> 
>


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