Re: [ovs-dev] [PATCH ovn] controller: Fall back to full recompute of pflow_output for vtep lport changes.

2021-09-17 Thread Numan Siddique
On Wed, Sep 15, 2021 at 8:12 PM Odintsov Vladislav wrote: > > Hi Numan, > > for me this patch solves the issue. > Actually, it’s a good point about adding test for ovn-controller-vtep to > eliminate this problem in future. > However there was another problem with HW VTEP support in OVN with using

Re: [ovs-dev] [PATCH ovn] controller: Fall back to full recompute of pflow_output for vtep lport changes.

2021-09-15 Thread Odintsov Vladislav
Hi Numan, for me this patch solves the issue. Actually, it’s a good point about adding test for ovn-controller-vtep to eliminate this problem in future. However there was another problem with HW VTEP support in OVN with using stateful services in the datapath. I’ve sent a patch series with a te

Re: [ovs-dev] [PATCH ovn] controller: Fall back to full recompute of pflow_output for vtep lport changes.

2021-09-08 Thread Numan Siddique
On Wed, Sep 8, 2021 at 5:59 AM Mark Gray wrote: > > On 07/09/2021 18:15, num...@ovn.org wrote: > > From: Numan Siddique > > > > When a vtep logical port changes, necessary flows are not added as > > expected. This is because the function physical_handle_flows_for_lport() > > in physical.c does n

Re: [ovs-dev] [PATCH ovn] controller: Fall back to full recompute of pflow_output for vtep lport changes.

2021-09-08 Thread Mark Gray
On 07/09/2021 18:15, num...@ovn.org wrote: > From: Numan Siddique > > When a vtep logical port changes, necessary flows are not added as > expected. This is because the function physical_handle_flows_for_lport() > in physical.c does not add flows required for vtep logical ports. > These flows ar

[ovs-dev] [PATCH ovn] controller: Fall back to full recompute of pflow_output for vtep lport changes.

2021-09-07 Thread numans
From: Numan Siddique When a vtep logical port changes, necessary flows are not added as expected. This is because the function physical_handle_flows_for_lport() in physical.c does not add flows required for vtep logical ports. These flows are added by physical_run(). So fall back to full recompu