diff options
| author | Ben Pfaff <blp@ovn.org> | 2017-01-19 20:39:26 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2017-01-20 14:10:35 -0800 |
| commit | bac295648d693edfe437d240cfb24a7c55ec169e (patch) | |
| tree | 439eb88f7f44646cf135078ccd86b4b19233aeea /tests | |
| parent | f1393f6f3469c25d22a7633c7b1fe0e8be94ed3d (diff) | |
| download | openvswitch-bac295648d693edfe437d240cfb24a7c55ec169e.tar.gz | |
actions: Make "arp { drop; };" acceptable.
Before this commit, the OVN action parser would accept "arp {};" and then
the formatter would format it back as "arp { drop; };", but the parser
didn't accept the latter. There were basically two choices: make the
parser accept "arp { drop; };" or make the formatter output "arp {};"
(or both). This patch does (only) the former, and adds a test to avoid
regression.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Mickey Spiegel <mickeys.dev@gmail.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/ovn.at | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/ovn.at b/tests/ovn.at index bc915c6f9..a514ebbda 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -871,6 +871,10 @@ ct_snat(); arp { eth.dst = ff:ff:ff:ff:ff:ff; output; }; output; encodes as controller(userdata=00.00.00.00.00.00.00.00.00.19.00.10.80.00.06.06.ff.ff.ff.ff.ff.ff.00.00.ff.ff.00.10.00.00.23.20.00.0e.ff.f8.40.00.00.00),resubmit(,64) has prereqs ip4 +arp { }; + formats as arp { drop; }; + encodes as controller(userdata=00.00.00.00.00.00.00.00) + has prereqs ip4 # get_arp get_arp(outport, ip4.dst); |