[ovs-dev] [PATCH] system-traffic: Use better names for OVS vxlan ports.

[Ben Pfaff]

On Tue, Apr 12, 2016 at 11:36:55AM -0700, Joe Stringer wrote:
> Signed-off-by: Joe Stringer <joe at ovn.org>
> ---
>  tests/system-traffic.at | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/system-traffic.at b/tests/system-traffic.at
> index 58212c1ed014..2e0dd9b4894a 100644
> --- a/tests/system-traffic.at
> +++ b/tests/system-traffic.at
> @@ -1401,7 +1401,7 @@ AT_CHECK([ip link set dev br-underlay up])
>  
>  dnl Set up tunnel endpoints on OVS outside the namespace and with a native
>  dnl linux device inside the namespace.
> -ADD_OVS_TUNNEL([vxlan], [br0], [at_ns0], [172.31.1.1], [10.1.1.100/24])
> +ADD_OVS_TUNNEL([vxlan], [br0], [at_vxlan0], [172.31.1.1], [10.1.1.100/24])
>  ADD_NATIVE_TUNNEL([vxlan], [at_vxlan1], [at_ns0], [172.31.1.100], [10.1.1.1/24],
>                    [id 0 dstport 4789])
>  
> @@ -1455,7 +1455,7 @@ AT_CHECK([ip link set dev br-underlay up])
>  
>  dnl Set up tunnel endpoints on OVS outside the namespace and with a native
>  dnl linux device inside the namespace.
> -ADD_OVS_TUNNEL([vxlan], [br0], [at_ns0], [172.31.1.1], ["fc00::2/96"])
> +ADD_OVS_TUNNEL([vxlan], [br0], [at_vxlan0], [172.31.1.1], ["fc00::2/96"])
>  ADD_NATIVE_TUNNEL([vxlan], [at_vxlan1], [at_ns0], [172.31.1.100], ["fc00::1/96"],
>                    [id 0 dstport 4789])

If I'm reading this right, then at_ns0 was being used as both a port
name and a network namespace name.  If that's right, then this seems
like a change likely to reduce confusion, so:
Acked-by: Ben Pfaff <blp at ovn.org>