You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Amogh Vasekar <am...@citrix.com> on 2013/12/31 03:30:05 UTC
Re: Review Request 15247: [DOC] CLOUDSTACK-4967: Add a section to describe
VNI allocation matter
> On Nov. 7, 2013, 4:32 p.m., Toshiaki Hatano wrote:
> > vxlan/en-US/plugin-vxlan-config-hypervisor.xml, line 30
> > <https://reviews.apache.org/r/15247/diff/1/?file=378377#file378377line30>
> >
> > This paragraph excluding line 32 is still valid after the patch.
> >
> > IMO it's better to explain why we need to open UDP/8472 port.
> > I also feel it's must to explain this plugin require multicast reachability between underlying interfaces, at least somewhere in this document.
> >
Reminder -
This review has been pending for long. Please mark the review as submitted if this has been committed already
Thanks!
- Amogh
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15247/#review28374
-----------------------------------------------------------
On Nov. 5, 2013, 9:31 p.m., Yoshikazu Nojima wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15247/
> -----------------------------------------------------------
>
> (Updated Nov. 5, 2013, 9:31 p.m.)
>
>
> Review request for cloudstack and Marcus Sorensen.
>
>
> Bugs: CLOUDSTACK-4967
> https://issues.apache.org/jira/browse/CLOUDSTACK-4967
>
>
> Repository: cloudstack-docs
>
>
> Description
> -------
>
> This patch updates CloudStack Plugin Guide for the VXLAN Plugin.
>
> - Add a section to describe VNIs allocation matter
> - Remove section that explain how to configure the bridge for traffic label because the bridge configuration is no longer necessary.
> - Update screen shots to catch up new UI visual appearance.
>
>
> Diffs
> -----
>
> vxlan/en-US/images/vxlan-trafficlabel.png 956d5f4f58f592ed3f260d6e1982c238c2ddf06a
> vxlan/en-US/images/vxlan-vniconfig.png PRE-CREATION
> vxlan/en-US/plugin-vxlan-config-hypervisor.xml 2c5e138
> vxlan/en-US/plugin-vxlan-config-management.xml 21f5461
> vxlan/en-US/plugin-vxlan-revision-history.xml ec04d11
>
> Diff: https://reviews.apache.org/r/15247/diff/
>
>
> Testing
> -------
>
> I generated and confirmed the PDF.
>
>
> Thanks,
>
> Yoshikazu Nojima
>
>