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