On 12 April 2016 at 21:15, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Apr 12, 2016 at 11:36:55AM -0700, Joe Stringer wrote: >> Signed-off-by: Joe Stringer <j...@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 <b...@ovn.org>
Yes, that was the intention (I could have explained that better in the commit message ;-) ) Applied to master, thanks. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev