Re: [ovs-dev] [PATCH ovn] nbctl: Execute unixctl_server_run() regardless of IDL state.

2021-01-19 Thread Numan Siddique
On Mon, Dec 21, 2020 at 9:28 PM Dumitru Ceara wrote: > This allows users to interact with the ovn-nbctl daemon even before the > first time the IDL client successfully connects to the server. > > Reported-at: > https://mail.openvswitch.org/pipermail/ovs-dev/2020-December/378826.html >

Re: [ovs-dev] RFC: stateless filter design

2021-01-19 Thread Numan Siddique
On Tue, Jan 19, 2021 at 11:07 AM Ihar Hrachyshka wrote: > Hi all, > > This email is primarily about choosing a preferred path to design API for > stateless filter support. There are also some related implementation > questions > at the bottom. > > === > > Problem: we need a way to configure

Re: [ovs-dev] [PATCH ovn branch-20.06 1/2] binding: Fix container port removal from local bindings.

2021-01-19 Thread Numan Siddique
On Tue, Jan 19, 2021 at 7:04 PM Dumitru Ceara wrote: > When the Port_Binding associated to a container port is removed make > sure we also remove it from the parent's 'children' shash. Container > ports don't have any VIFs associated so it's safe to destroy the > container port local binding

[ovs-dev] Why does VXLAN tunnel need to wait for a long time to work normally in OVS DPDK?

2021-01-19 Thread 杨燚
Hi, folks I noticed ping from local IP to remote IP of VXLAN port can’t work immediately after OVS DPDK is started and VXLAN port is created, sometimes it needs to wait for more than 1 minute, I found ARP request triggered by ping and ARP request sent by ovs itself can’t be responded

Re: [ovs-dev] [PATCH ovs] conntrack: Update the icmp stats accurately.

2021-01-19 Thread Yi-Hung Wei
On Tue, Jan 19, 2021 at 6:45 AM Aaron Conole wrote: > > The fixes tag for this should be: > > Fixes: a867c010ee91 ("conntrack: Fix conntrack new state") Yes, commit a867c010ee91 ("conntrack: Fix conntrack new state") is the right one to fix. Commit a867c010ee91 only takes care of TCP, UDP

Re: [ovs-dev] [PATCH ovn] ovn-nbctl: add ecmp/ecmp-symmetric-reply to lr-route-list command

2021-01-19 Thread Mark Michelson
Excellent. I tried to make it angry but it remained calm no matter what I threw at it. The only "flaw" is that if you add just one route with "--ecmp" then it will not display "ecmp" when you list the route. ovn-northd doesn't do any special ECMP handling with this single route, either, so the

Re: [ovs-dev] [PATCH ovs] conntrack: Update the icmp stats accurately.

2021-01-19 Thread Aaron Conole
Aaron Conole writes: > Tonghao Zhang writes: > >> On Fri, Jan 15, 2021 at 5:20 AM Aaron Conole wrote: >>> >>> Hi Tonghao Xhang, >>> >>> xiangxia.m@gmail.com writes: >>> >>> > From: Tonghao Zhang >>> > >>> > The same icmp packet may traverse conntrack module more than once. >>> > Or same

Re: [ovs-dev] [PATCH ovs] conntrack: Update the icmp stats accurately.

2021-01-19 Thread Aaron Conole
Tonghao Zhang writes: > On Fri, Jan 15, 2021 at 5:20 AM Aaron Conole wrote: >> >> Hi Tonghao Xhang, >> >> xiangxia.m@gmail.com writes: >> >> > From: Tonghao Zhang >> > >> > The same icmp packet may traverse conntrack module more than once. >> > Or same icmp packets traverse contranck

[ovs-dev] [PATCH ovn branch-20.06 2/2] binding: Always delete child port bindings first.

2021-01-19 Thread Dumitru Ceara
When incrementally processing changes, child Port Bindings (container and virtual ports) must be deleted first, before their parents, because they need to be removed from their parent's children hash to avoid parents with children with NULL port bindings. Signed-off-by: Dumitru Ceara

[ovs-dev] [PATCH ovn branch-20.06 1/2] binding: Fix container port removal from local bindings.

2021-01-19 Thread Dumitru Ceara
When the Port_Binding associated to a container port is removed make sure we also remove it from the parent's 'children' shash. Container ports don't have any VIFs associated so it's safe to destroy the container port local binding when the SB.Port_Binding is deleted. Signed-off-by: Dumitru

Re: [ovs-dev] [External] : Re: Fwd: tracing ovs flows in br-int

2021-01-19 Thread Brendan Doyle
Thanks On 19/01/2021 12:14, Ilya Maximets wrote: On 1/18/21 7:38 PM, Brendan Doyle wrote: Forwarded Message Subject: tracing ovs flows in br-int Date: Mon, 18 Jan 2021 18:31:53 + From: Brendan Doyle Organization: Oracle Corporation To:

Re: [ovs-dev] Fwd: tracing ovs flows in br-int

2021-01-19 Thread Ilya Maximets
On 1/18/21 7:38 PM, Brendan Doyle wrote: > > > > Forwarded Message > Subject: tracing ovs flows in br-int > Date: Mon, 18 Jan 2021 18:31:53 + > From: Brendan Doyle > Organization: Oracle Corporation > To: ovs-disc...@openvswitch.org > > > > Hi

Re: [ovs-dev] [PATCH ovn 0/2] binding: Fix incremental processing of child Port_Bindings.

2021-01-19 Thread Numan Siddique
On Tue, Jan 19, 2021 at 3:03 PM Dumitru Ceara wrote: > On 1/19/21 10:27 AM, Numan Siddique wrote: > > > > > > On Tue, Jan 19, 2021 at 2:43 PM Dumitru Ceara > > wrote: > > > > On 1/19/21 8:32 AM, Numan Siddique wrote: > > > > > > > > > On Mon, Jan 18,

[ovs-dev] [PATCH] system-dpdk: Fix 4th and 5th testcases of the system-dpdk testsuite.

2021-01-19 Thread Emma Finn
Testpmd has been renamed to dpdk-testpmd as of DPDK 20.11. This commit resolves this issue and fixes these tests. Signed-off-by: Emma Finn --- tests/system-dpdk.at | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/system-dpdk.at b/tests/system-dpdk.at index

Re: [ovs-dev] [PATCH ovn 0/2] binding: Fix incremental processing of child Port_Bindings.

2021-01-19 Thread Dumitru Ceara
On 1/19/21 10:27 AM, Numan Siddique wrote: On Tue, Jan 19, 2021 at 2:43 PM Dumitru Ceara > wrote: On 1/19/21 8:32 AM, Numan Siddique wrote: > > > On Mon, Jan 18, 2021 at 10:20 PM Dumitru Ceara mailto:dce...@redhat.com> >

Re: [ovs-dev] [PATCH ovn 0/2] binding: Fix incremental processing of child Port_Bindings.

2021-01-19 Thread Numan Siddique
On Tue, Jan 19, 2021 at 2:43 PM Dumitru Ceara wrote: > On 1/19/21 8:32 AM, Numan Siddique wrote: > > > > > > On Mon, Jan 18, 2021 at 10:20 PM Dumitru Ceara > > wrote: > > > > Dumitru Ceara (2): > >binding: Fix container port removal from local bindings.

Re: [ovs-dev] [PATCH ovn 0/2] binding: Fix incremental processing of child Port_Bindings.

2021-01-19 Thread Dumitru Ceara
On 1/19/21 8:32 AM, Numan Siddique wrote: On Mon, Jan 18, 2021 at 10:20 PM Dumitru Ceara > wrote: Dumitru Ceara (2):       binding: Fix container port removal from local bindings.       binding: Always delete child port bindings first. Thanks Dumitru