From 4c15a9ade8254ef9a0a3e1b60b953e5355a5e797 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Sun, 23 Oct 2016 10:50:35 -0700 Subject: tests: Fix order confusion in "ovn -- 2 HVs, 4 lports/HV, localnet ports". The order of src and dst was swapped both in assignment and reference, which meant that the result worked OK but was really confusing to try to extend or modify. Signed-off-by: Ben Pfaff Acked-by: Russell Bryant --- tests/ovn.at | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/ovn.at b/tests/ovn.at index a226849fa..89742e57f 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -1598,8 +1598,8 @@ for i in 1 2; do done done test_packet() { - local inport=$1 src=$2 dst=$3 eth=$4; shift; shift; shift; shift - local packet=${src}${dst}${eth} + local inport=$1 dst=$2 src=$3 eth=$4; shift; shift; shift; shift + local packet=${dst}${src}${eth} hv=`vif_to_hv $inport` vif=vif$inport as $hv ovs-appctl netdev-dummy/receive $vif $packet -- cgit v1.2.1