diff mbox

[ovs-dev,22/31] fixup: Adapt tests to revised policy.

Message ID 20170612222856.14108-23-blp@ovn.org
State Superseded
Headers show

Commit Message

Ben Pfaff June 12, 2017, 10:28 p.m. UTC
Since packet_type is now accepted on any bridge, there's no need to test
for it being rejected.

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 tests/packet-type-aware.at | 59 +++++-----------------------------------------
 1 file changed, 6 insertions(+), 53 deletions(-)

Comments

Jan Scheurich June 14, 2017, 10:50 p.m. UTC | #1
Agreed.

> -----Original Message-----
> From: ovs-dev-bounces@openvswitch.org [mailto:ovs-dev-bounces@openvswitch.org] On Behalf Of Ben Pfaff
> Sent: Tuesday, 13 June, 2017 00:29
> To: dev@openvswitch.org
> Cc: Ben Pfaff <blp@ovn.org>
> Subject: [ovs-dev] [PATCH 22/31] fixup: Adapt tests to revised policy.
> 
> Since packet_type is now accepted on any bridge, there's no need to test
> for it being rejected.
> 
> Signed-off-by: Ben Pfaff <blp@ovn.org>
> ---
>  tests/packet-type-aware.at | 59 +++++-----------------------------------------
>  1 file changed, 6 insertions(+), 53 deletions(-)
> 
> diff --git a/tests/packet-type-aware.at b/tests/packet-type-aware.at
> index 2c760bfa17db..cfa207893b3b 100644
> --- a/tests/packet-type-aware.at
> +++ b/tests/packet-type-aware.at
> @@ -1,55 +1,8 @@
>  AT_BANNER([packet-type-aware pipeline])
> 
> -AT_SETUP([ptap - create packet-type-aware bridge])
> -
> -OVS_VSWITCHD_START([])
> -
> -AT_CHECK([
> -    ovs-vsctl set bridge br0 \
> -    datapath_type=dummy \
> -    other-config:packet-type-aware=true
> -], [0])
> -
> -AT_CHECK([ovs-vsctl list bridge br0 | grep other_config], [0], [dnl
> -other_config        : {datapath-id="fedcba9876543210", hwaddr="aa:55:aa:55:00:00", packet-type-aware="true"}
> -])
> -
> -AT_CHECK([ovs-ofctl -Oopenflow13 dump-table-features br0 | grep packet_type], [0], [dnl
> -      packet_type: exact match or wildcard
> -])
> -AT_CHECK([ovs-ofctl -Oopenflow14 dump-table-features br0 | grep packet_type], [0], [dnl
> -      packet_type: exact match or wildcard
> -])
> -AT_CHECK([ovs-ofctl -Oopenflow15 dump-table-features br0 | grep packet_type], [0], [dnl
> -      packet_type: exact match or wildcard
> -])
> -
> -AT_CHECK([
> -    ovs-vsctl set bridge br0 other-config:packet-type-aware=false
> -], [0])
> -
> -AT_CHECK([ovs-vsctl list bridge br0 | grep other_config], [0], [dnl
> -other_config        : {datapath-id="fedcba9876543210", hwaddr="aa:55:aa:55:00:00", packet-type-aware="false"}
> -])
> -
> -AT_CHECK([ovs-ofctl -Oopenflow13 dump-table-features br0 | grep packet_type], [1])
> -AT_CHECK([ovs-ofctl -Oopenflow14 dump-table-features br0 | grep packet_type], [1])
> -AT_CHECK([ovs-ofctl -Oopenflow15 dump-table-features br0 | grep packet_type], [1])
> -
> -OVS_VSWITCHD_STOP
> -AT_CLEANUP
> -
> -
>  AT_SETUP([ptap - legal flow entries in ptap bridge])
> 
> -OVS_VSWITCHD_START([])
> -
> -AT_CHECK([
> -    ovs-vsctl set bridge br0 \
> -    datapath_type=dummy \
> -    protocols=OpenFlow10,OpenFlow13,OpenFlow14,OpenFlow15 \
> -    other-config:packet-type-aware=true
> -], [0])
> +OVS_VSWITCHD_START
> 
>  AT_CHECK([
>      ovs-ofctl del-flows br0
> @@ -64,11 +17,11 @@ AT_CHECK([
>  ], [0])
> 
>  AT_CHECK([ovs-ofctl -Oopenflow15 dump-flows br0 | ofctl_strip | sort | grep actions], [0], [dnl
> - priority=1,packet_type=(0,0),dl_src=11:22:33:44:55:66,dl_type=0x1234 actions=drop
> - priority=1,packet_type=(0,0),ip,nw_dst=10.11.12.13 actions=drop
> - priority=1,packet_type=(0,0),tcp6 actions=drop
> - priority=2,packet_type=(0,0),arp,arp_tpa=10.11.12.13 actions=drop
> - priority=2,packet_type=(0,0),dl_src=11:22:33:44:55:66,dl_type=0x4567 actions=drop
> + priority=1,dl_src=11:22:33:44:55:66,dl_type=0x1234 actions=drop
> + priority=1,ip,nw_dst=10.11.12.13 actions=drop
> + priority=1,tcp6 actions=drop
> + priority=2,arp,arp_tpa=10.11.12.13 actions=drop
> + priority=2,dl_src=11:22:33:44:55:66,dl_type=0x4567 actions=drop
>   priority=3,packet_type=(1,0x800),nw_dst=10.11.12.13 actions=drop
>   priority=3,packet_type=(1,0x806),arp_tpa=10.11.12.13 actions=drop
>   priority=3,packet_type=(1,0x86dd),ipv6_dst=1234:5678::/32 actions=drop
> --
> 2.10.2
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
diff mbox

Patch

diff --git a/tests/packet-type-aware.at b/tests/packet-type-aware.at
index 2c760bfa17db..cfa207893b3b 100644
--- a/tests/packet-type-aware.at
+++ b/tests/packet-type-aware.at
@@ -1,55 +1,8 @@ 
 AT_BANNER([packet-type-aware pipeline])
 
-AT_SETUP([ptap - create packet-type-aware bridge])
-
-OVS_VSWITCHD_START([])
-
-AT_CHECK([
-    ovs-vsctl set bridge br0 \
-    datapath_type=dummy \
-    other-config:packet-type-aware=true
-], [0])
-
-AT_CHECK([ovs-vsctl list bridge br0 | grep other_config], [0], [dnl
-other_config        : {datapath-id="fedcba9876543210", hwaddr="aa:55:aa:55:00:00", packet-type-aware="true"}
-])
-
-AT_CHECK([ovs-ofctl -Oopenflow13 dump-table-features br0 | grep packet_type], [0], [dnl
-      packet_type: exact match or wildcard
-])
-AT_CHECK([ovs-ofctl -Oopenflow14 dump-table-features br0 | grep packet_type], [0], [dnl
-      packet_type: exact match or wildcard
-])
-AT_CHECK([ovs-ofctl -Oopenflow15 dump-table-features br0 | grep packet_type], [0], [dnl
-      packet_type: exact match or wildcard
-])
-
-AT_CHECK([
-    ovs-vsctl set bridge br0 other-config:packet-type-aware=false
-], [0])
-
-AT_CHECK([ovs-vsctl list bridge br0 | grep other_config], [0], [dnl
-other_config        : {datapath-id="fedcba9876543210", hwaddr="aa:55:aa:55:00:00", packet-type-aware="false"}
-])
-
-AT_CHECK([ovs-ofctl -Oopenflow13 dump-table-features br0 | grep packet_type], [1])
-AT_CHECK([ovs-ofctl -Oopenflow14 dump-table-features br0 | grep packet_type], [1])
-AT_CHECK([ovs-ofctl -Oopenflow15 dump-table-features br0 | grep packet_type], [1])
-
-OVS_VSWITCHD_STOP
-AT_CLEANUP
-
-
 AT_SETUP([ptap - legal flow entries in ptap bridge])
 
-OVS_VSWITCHD_START([])
-
-AT_CHECK([
-    ovs-vsctl set bridge br0 \
-    datapath_type=dummy \
-    protocols=OpenFlow10,OpenFlow13,OpenFlow14,OpenFlow15 \
-    other-config:packet-type-aware=true
-], [0])
+OVS_VSWITCHD_START
 
 AT_CHECK([
     ovs-ofctl del-flows br0
@@ -64,11 +17,11 @@  AT_CHECK([
 ], [0])
 
 AT_CHECK([ovs-ofctl -Oopenflow15 dump-flows br0 | ofctl_strip | sort | grep actions], [0], [dnl
- priority=1,packet_type=(0,0),dl_src=11:22:33:44:55:66,dl_type=0x1234 actions=drop
- priority=1,packet_type=(0,0),ip,nw_dst=10.11.12.13 actions=drop
- priority=1,packet_type=(0,0),tcp6 actions=drop
- priority=2,packet_type=(0,0),arp,arp_tpa=10.11.12.13 actions=drop
- priority=2,packet_type=(0,0),dl_src=11:22:33:44:55:66,dl_type=0x4567 actions=drop
+ priority=1,dl_src=11:22:33:44:55:66,dl_type=0x1234 actions=drop
+ priority=1,ip,nw_dst=10.11.12.13 actions=drop
+ priority=1,tcp6 actions=drop
+ priority=2,arp,arp_tpa=10.11.12.13 actions=drop
+ priority=2,dl_src=11:22:33:44:55:66,dl_type=0x4567 actions=drop
  priority=3,packet_type=(1,0x800),nw_dst=10.11.12.13 actions=drop
  priority=3,packet_type=(1,0x806),arp_tpa=10.11.12.13 actions=drop
  priority=3,packet_type=(1,0x86dd),ipv6_dst=1234:5678::/32 actions=drop