summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Pfaff <blp@ovn.org>2016-10-23 10:50:35 -0700
committerBen Pfaff <blp@ovn.org>2016-12-06 09:46:02 -0800
commit4c15a9ade8254ef9a0a3e1b60b953e5355a5e797 (patch)
tree211e267d4588a5dca40b1abe3fdffaa790ad666f
parent569c26da9d0f93bc0e110ee267d4f3a770a0da0c (diff)
downloadopenvswitch-4c15a9ade8254ef9a0a3e1b60b953e5355a5e797.tar.gz
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 <blp@ovn.org> Acked-by: Russell Bryant <russell@ovn.org>
-rw-r--r--tests/ovn.at4
1 files 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