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> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
