mbox series

[v2,net-next,0/5] selftests: openvswitch: add flow programming cases

Message ID 20230728115940.578658-1-aconole@redhat.com
Headers
Series selftests: openvswitch: add flow programming cases |

Message

Aaron Conole July 28, 2023, 11:59 a.m. UTC 
  The openvswitch selftests currently contain a few cases for managing the
datapath, which includes creating datapath instances, adding interfaces,
and doing some basic feature / upcall tests.  This is useful to validate
the control path.

Add the ability to program some of the more common flows with actions. This
can be improved overtime to include regression testing, etc.

Changes from original:

1. Fix issue when parsing ipv6 in the NAT action
2. Fix issue calculating length during ctact parsing
3. Fix error message when invalid bridge is passed
4. Fold in Adrian's patch to support key masks

Aaron Conole (4):
  selftests: openvswitch: add an initial flow programming case
  selftests: openvswitch: add a test for ipv4 forwarding
  selftests: openvswitch: add basic ct test case parsing
  selftests: openvswitch: add ct-nat test case with ipv4

Adrian Moreno (1):
  selftests: openvswitch: support key masks

 .../selftests/net/openvswitch/openvswitch.sh  | 223 +++++++
 .../selftests/net/openvswitch/ovs-dpctl.py    | 601 +++++++++++++++++-
 2 files changed, 800 insertions(+), 24 deletions(-)

Comments

Simon Horman July 30, 2023, 3:42 p.m. UTC | #1 
On Fri, Jul 28, 2023 at 07:59:37AM -0400, Aaron Conole wrote:
> From: Adrian Moreno <amorenoz@redhat.com>
> 
> From: Adrian Moreno <amorenoz@redhat.com>
> 
> The default value for the mask actually depends on the value (e.g: if
> the value is non-null, the default is full-mask), so change the convert
> functions to accept the full, possibly masked string and let them figure
> out how to parse the differnt values.

nit: differnt -> different