[ovs-dev] [PATCH v2] ovs-lib: pass optional --election-timer arg to ovsdb-tool

2021-06-07 Thread Dan Williams
Signed-off-by: Dan Williams --- v2: put --election-timer arg before create-cluster per Ilya  utilities/ovs-lib.in | 10 --  1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in index ab38ece458b7b..61a062fa992da 100644 ---

Re: [ovs-dev] [PATCH ovn 3/3] Honor ACL direction when omitting ct for stateless

2021-06-07 Thread Ihar Hrachyshka
On Wed, Jun 2, 2021 at 12:22 AM Han Zhou wrote: > > > > On Tue, Jun 1, 2021 at 12:28 PM Ihar Hrachyshka wrote: > > > > On Thu, May 20, 2021 at 9:55 PM Han Zhou wrote: > > > > > > > > > > > > On Thu, May 20, 2021 at 3:22 PM Han Zhou wrote: > > > > > > > > > > > > > > > > On Mon, May 17, 2021 at

[ovs-dev] [PATCH] Remove Python 2 leftovers.

2021-06-07 Thread Rosemarie O'Riorden
Fixes: 1ca0323e7c29 ("Require Python 3 and remove support for Python 2.") Reported at: https://bugzilla.redhat.com/show_bug.cgi?id=1949875 Signed-off-by: Rosemarie O'Riorden --- python/ovstest/tests.py | 4 +--- python/ovstest/util.py | 2 +- python/setup.py

Re: [ovs-dev] [PATCH v3 0/9] Stream Record/Replay.

2021-06-07 Thread Ilya Maximets
On 6/4/21 4:40 PM, Dumitru Ceara wrote: > On 5/27/21 3:28 PM, Ilya Maximets wrote: >> This patch set adds new stream provider and other functionality in >> order to record all the incoming data on all the steams (ssl, tcp, >> unixctl) of openvswitch library based applications and replay these >>

Re: [ovs-dev] [PATCH] ovs-lib: pass optional --election-timer arg to ovsdb-tool

2021-06-07 Thread Dan Williams
On Mon, 2021-06-07 at 20:56 +0200, Ilya Maximets wrote: > On 6/7/21 6:04 PM, Dan Williams wrote: > > > > Signed-off-by: Dan Williams > > --- > >  utilities/ovs-lib.in | 10 -- > >  1 file changed, 8 insertions(+), 2 deletions(-) > > > > diff --git a/utilities/ovs-lib.in

Re: [ovs-dev] [PATCH] ovs-lib: pass optional --election-timer arg to ovsdb-tool

2021-06-07 Thread Ilya Maximets
On 6/7/21 6:04 PM, Dan Williams wrote: > > Signed-off-by: Dan Williams > --- > utilities/ovs-lib.in | 10 -- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in > index ab38ece458b7b..d5aef8c85b3e0 100644 > ---

[ovs-dev] [PATCH v4] netlink: removed incorrect optimization

2021-06-07 Thread Toms Atteka
This optimization caused FLOW_TNL_F_UDPIF flag not to be used in hash calculation for geneve tunnel when revalidating flows which resulted in different cache hash values and incorrect behaviour. Added test to prevent regression. CC: Jesse Gross Fixes: 6728d578f64e ("dpif-netdev: Translate

Re: [ovs-dev] [PATCH v2] Fix redundant datapath set ethernet action with NSH Decap

2021-06-07 Thread Jan Scheurich via dev
> -Original Message- > From: Martin Varghese > Sent: Monday, 7 June, 2021 16:47 > To: Ilya Maximets > Cc: d...@openvswitch.org; echau...@redhat.com; Jan Scheurich > ; Martin Varghese > > Subject: Re: [ovs-dev] [PATCH v2] Fix redundant datapath set ethernet action > with NSH Decap > >

[ovs-dev] [PATCH] ovs-lib: pass optional --election-timer arg to ovsdb-tool

2021-06-07 Thread Dan Williams
Signed-off-by: Dan Williams --- utilities/ovs-lib.in | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in index ab38ece458b7b..d5aef8c85b3e0 100644 --- a/utilities/ovs-lib.in +++ b/utilities/ovs-lib.in @@ -495,15 +495,21 @@

Re: [ovs-dev] [ovs-discuss] Patch for PACKET_OUT getting deleted twice crash

2021-06-07 Thread Ilya Maximets
On 6/7/21 3:59 PM, Ilya Maximets wrote: > On 6/7/21 3:09 PM, Aaron Conole wrote: >> Ilya Maximets writes: >> Here is a patch with both a test and a fix. >> >> Thanks so much! It's nice to get fixes, but I think it's really great >> when test cases come along with them. >> >>> Hi. Thanks

Re: [ovs-dev] [PATCH v2] Fix redundant datapath set ethernet action with NSH Decap

2021-06-07 Thread Martin Varghese
On Wed, May 19, 2021 at 12:26:40PM +0200, Ilya Maximets wrote: > On 5/19/21 5:26 AM, Martin Varghese wrote: > > On Tue, May 18, 2021 at 10:03:39PM +0200, Ilya Maximets wrote: > >> On 5/17/21 3:45 PM, Martin Varghese wrote: > >>> From: Martin Varghese > >>> > >>> When a decap action is applied on

Re: [ovs-dev] [ovs-discuss] Patch for PACKET_OUT getting deleted twice crash

2021-06-07 Thread Ilya Maximets
On 6/7/21 3:09 PM, Aaron Conole wrote: > Ilya Maximets writes: > >>> Here is a patch with both a test and a fix. > > Thanks so much! It's nice to get fixes, but I think it's really great > when test cases come along with them. > >> Hi. Thanks for working n this! >> >> CC: ovs-dev >> >>> Not

Re: [ovs-dev] [ovs-discuss] Patch for PACKET_OUT getting deleted twice crash

2021-06-07 Thread Aaron Conole
Ilya Maximets writes: >> Here is a patch with both a test and a fix. Thanks so much! It's nice to get fixes, but I think it's really great when test cases come along with them. > Hi. Thanks for working n this! > > CC: ovs-dev > >> Not submitting as a formal >> patch because I would like some

[ovs-dev] [PATCH ovn 1/2] tests: Add debugging 4 HV, 3LS, 2 LR test.

2021-06-07 Thread Mark Michelson
Add some checking to "4 HV, 3 LS, 2 LR, packet test with HA distributed router gateway port". Signed-off-by: Mark Michelson --- tests/ovn.at | 90 1 file changed, 48 insertions(+), 42 deletions(-) diff --git a/tests/ovn.at b/tests/ovn.at

[ovs-dev] [PATCH ovn 2/2] tests: Fix consistency of 4 HV, 3LS, 2 LR test runs.

2021-06-07 Thread Mark Michelson
Much like the 4 HV, 1 LS, 1 LR test, this one was also having issues where the pcap file would not be reset before attempting to capture packets. Therefore, the pcap file would be reset after having captured packets we were expecting to see. Therefore, this patch does similarly to 879ebc8c6, in

[ovs-dev] [PATCH] dpif-netdev: Report overhead busy cycles per pmd.

2021-06-07 Thread David Marchand
When setting PMD auto load balance parameters and observing the feature (and the per rxq statistics) in action, you can easily get rebalances while the sum of per rxq pmd usage is below the pmd load threshold you had set. This is because the dpif-netdev/pmd-rxq-show command only reports "pure"

Re: [ovs-dev] [PATCH v2] ofproto-dpif-upcall: Improve concurrency by adjust flow-limit

2021-06-07 Thread taoyunupt
Hi , Please help to review this patch. It has passed couple of days. Hope to know your idea. Thanks. Thanks, YUN At 2021-05-26 10:44:16, "Tao YunXiang" wrote: >OVS uses a set of gentle mechanisms to control the number of flows in >datapath. However, it is difficult for the

Re: [ovs-dev] [PATCH] system-traffic.at:add missing comma

2021-06-07 Thread taoyunupt
Hi , Please help to review this patch. It has passed couple of days. Thanks. Thanks, YUN At 2021-05-26 10:53:45, "Tao YunXiang" wrote: >Add missing comma. > >Signed-off-by: Tao YunXiang >Cc: Joe Stringer > >--- > tests/system-traffic.at | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [ovs-dev] [PATCH 1/2] bond: Fix broken rebalancing after link state changes.

2021-06-07 Thread Ilya Maximets
On 8/2/17 11:09 PM, Andy Zhou wrote: > On Thu, Jul 20, 2017 at 10:21 AM, Ilya Maximets > wrote: >> There are 3 constraints for moving hashes from one slave to another: >> >> 1. The load difference is larger than ~3% of one slave's load. >> 2. The load difference between slaves

Re: [ovs-dev] [PATCH ovn] ovn-trace: correctly handle ct_dnat(IP) action

2021-06-07 Thread Dumitru Ceara
On 6/4/21 7:06 PM, Mark Gray wrote: > ovn-trace does not set translated ip address for ct_dnat() > actions when tracing. This causes the trace to end prematurely. Hi Mark, Thanks for the patch! > > This can be tested with the following or an equivalent for IPv6: > > ovn-nbctl ls-add sw0 >

Re: [ovs-dev] [PATCH ovn 0/4] Fix ovn-controller I-P for multicast groups and lport changes

2021-06-07 Thread Dumitru Ceara
On 6/7/21 7:33 AM, Han Zhou wrote: > On Fri, Jun 4, 2021 at 6:56 AM Dumitru Ceara wrote: >> >> On 5/28/21 9:23 PM, Han Zhou wrote: >>> The series fixes incremental processing for missing dependency handling > for >>> multicast group and logical port binding changes when computing logical > flows.