[ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

Ben Pfaff blp at ovn.org
Mon Jul 4 04:52:44 UTC 2016


On Sun, Jul 03, 2016 at 08:17:32PM -0700, Guru Shetty wrote:
> On 3 July 2016 at 10:24, Ben Pfaff <blp at ovn.org> wrote:
> 
> > On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote:
> > > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table.
> > > For packets that need to be load balanced, this table sets reg0[0]
> > > to act as a hint for the pre-stateful table to send the packet to
> > > the conntrack table for defragmentation.

...

> > Is there a way to test this?
> 
> The load balancing itself via group actions are tested here:
> https://github.com/openvswitch/ovs/blob/master/tests/system-traffic.at#L2529
> 
> But OVN tests do not exist end to end as unit test framework does not
> integrate with conntrack NAT.
> One option is to wait for Daniele's userpace NAT work to get merged to add
> the tests.
>
> One way that we could test it is to look for specific group flows that get
> generated with 'ovs-ofctl dump-groups' without any traffic sent. I can
> create a unit test for that as a separate patch.

That might be useful; I'll leave it to you to decide.

> Thank you, I applied it with the following incremental around documentation
> (which I had completely forgotten about, sorry about that. I am happy to
> send any incremental documentation fixes if you have any comments. I
> thought about sending another version, but I would likely again have
> conflict with Numan's DHCP patches).

Thanks for adding the documentation.



More information about the dev mailing list