[ovs-dev] [PATCH ovn] ovn-nbctl: Add some hint for --ecmp in error message.

2020-06-19 Thread Han Zhou
Add the hint to remind user about --ecmp option when needed. Signed-off-by: Han Zhou --- tests/ovn-nbctl.at| 2 +- utilities/ovn-nbctl.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/ovn-nbctl.at b/tests/ovn-nbctl.at index 14de1a8..54b4ad2 100644 ---

[ovs-dev] Demandas - Acciones preventivas y de negociación

2020-06-19 Thread Terminacion de la relacion laboral
Buenos día Quise aprovechar la oportunidad de hacerte una invitación para tomar nuestro curso de 4 horas : Nombre: Mediación de conflictos y prevención de demandas laborales. Fecha: Miércoles 8 de Julio Horario: 10:00 am a 14:00 pm Formato: En línea con interacción en vivo. Lugar: En Vivo

[ovs-dev] [PATCH v2] tc: Changes to netlink message population for OVS-TC Flower Offload

2020-06-19 Thread dsatish
From: Satish Dhote OVS-TC offload sends only fields that are not completely masked out to the TC kernel module. Some of the hardware we work with requires all fields extracted from the packet along with the mask to be sent to the driver. Hardware may optimize tables based on entire fields of

Re: [ovs-dev] [PATCH v7 ovn] Add support for DHCP domain search option (119)

2020-06-19 Thread 0-day Robot
Bleep bloop. Greetings Ankur Sharma, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or

[ovs-dev] [PATCH v7 ovn] Add support for DHCP domain search option (119)

2020-06-19 Thread Ankur Sharma
From: Dhathri Purohith Domain search list is encoded according to the specifications in RFC 1035, section 4.1.4. Signed-off-by: Dhathri Purohith Signed-off-by: Ankur Sharma --- lib/actions.c | 90 - lib/ovn-l7.h| 3 ++

[ovs-dev] FINALE NOTIFICATION FUER DIE AUSZAHLUNG IHRES GEWINNES

2020-06-19 Thread FINALE NOTIFICATION FUER DIE AUSZAHLUNG IHRES GEWINNES
Sehr geehrter Begьnstigter, Wir mцchten Sie ьber Ihren Gewinnbetrag informieren,Bitte цffnen Sie den Anhang, um es zu sehen. Anbei erhalten sie die Finale Notification fuer die Auszahlung ihres Gewinnes. Setzen sie sich bitte Umgehend mit uns in Verbindung,damit wir den Auazahlungsprozess

[ovs-dev] [PATCH v2] ovs-dpctl-top: Skip "eth()" element

2020-06-19 Thread Timothy Redaelli
With commit efde188622ae ("odp-util: Print eth() for Ethernet flows if packet_type is absent.") "eth()" is printed for Ethernet flows if packet_type is absent, but this broke "ovs-dpctl-top" since it expects that every element has a value. This commit skips the parsing of the empty "eth()"

[ovs-dev] [PATCH v2] ovs-dpctl-top: Open stdin in binary mode

2020-06-19 Thread Timothy Redaelli
On Python3 buffering can only be disabled on files opened in binary mode. Signed-off-by: Timothy Redaelli --- v2: - Added missing Signed-off-by utilities/ovs-dpctl-top.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-dpctl-top.in b/utilities/ovs-dpctl-top.in

[ovs-dev] ATTN: Check Your Accounts

2020-06-19 Thread Grace Nguyen
Image of your report For: Date: June 19th, 2020 Due to Equifax's latest security breach, your current TransUnion, Equifax, and Experian scores may have changed. Your Scores are available now at no charge. View Your Score Unsubscribe To stop receiving emails from us please click

Re: [ovs-dev] [PATCH] tc: Changes to netlink message population for OVS-TC Flower Offload

2020-06-19 Thread 0-day Robot
Bleep bloop. Greetings dsatish, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author dsatish needs to sign off. WARNING: Unexpected sign-offs from developers who are not

Re: [ovs-dev] [PATCH ovn v13 0/7] Incremental processing improvements.

2020-06-19 Thread Numan Siddique
On Fri, Jun 19, 2020 at 6:09 PM Dumitru Ceara wrote: > On 6/19/20 1:09 PM, num...@ovn.org wrote: > > From: Numan Siddique > > > > This patch series handles port binding, datapath binding, ovs interface > changes, > > runtime data changes, sb chassis changes incrementally. > > > > > > v12 -> v13

Re: [ovs-dev] [PATCH ovn v13 0/7] Incremental processing improvements.

2020-06-19 Thread Dumitru Ceara
On 6/19/20 1:09 PM, num...@ovn.org wrote: > From: Numan Siddique > > This patch series handles port binding, datapath binding, ovs interface > changes, > runtime data changes, sb chassis changes incrementally. > > > v12 -> v13 > - > * Addressed the review comments from Dumitru. > *

Re: [ovs-dev] [PATCH ovn v13 3/7] ovn-controller: Handle runtime data changes in flow output engine

2020-06-19 Thread Numan Siddique
On Fri, Jun 19, 2020 at 5:22 PM Dumitru Ceara wrote: > On 6/19/20 1:10 PM, num...@ovn.org wrote: > > From: Numan Siddique > > > > In order to handle runtime data changes incrementally, the flow outut > > runtime data handle should know the changed runtime data. > > Runtime data now tracks the

Re: [ovs-dev] [PATCH ovn v13 5/7] tests: Enhance ovn-performance testing by adding gw router port.

2020-06-19 Thread Numan Siddique
On Fri, Jun 19, 2020 at 5:57 PM Dumitru Ceara wrote: > On Fri, Jun 19, 2020 at 1:11 PM wrote: > > > > From: Numan Siddique > > > > This covers the scenario of setting up/deleting of BFD tunnels > > for HA. > > > > Acked-by: Han Zhou > > Signed-off-by: Numan Siddique > > --- > >

Re: [ovs-dev] [PATCH] ovs-dpctl-top: open stdin in binary mode

2020-06-19 Thread 0-day Robot
Bleep bloop. Greetings Timothy Redaelli, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Timothy Redaelli needs to sign off. Lines checked: 27, Warnings: 0, Errors:

Re: [ovs-dev] [PATCH ovn v13 5/7] tests: Enhance ovn-performance testing by adding gw router port.

2020-06-19 Thread Dumitru Ceara
On Fri, Jun 19, 2020 at 1:11 PM wrote: > > From: Numan Siddique > > This covers the scenario of setting up/deleting of BFD tunnels > for HA. > > Acked-by: Han Zhou > Signed-off-by: Numan Siddique > --- > tests/ovn-performance.at | 106 +++ > 1 file changed,

Re: [ovs-dev] [PATCH 0/7] user defined events and notification

2020-06-19 Thread Gowrishankar Muthukrishnan
Hi, Just curious to probe you again if you had a chance to check this feature. Thanks in advance. On Tue, Jun 2, 2020 at 3:40 PM Gowrishankar Muthukrishnan < gmuth...@redhat.com> wrote: > This patchset enables new feature "user defined events and notification". > In short, by this feature, one

[ovs-dev] [PATCH] tc: Changes to netlink message population for OVS-TC Flower Offload

2020-06-19 Thread dsatish
OVS-TC offload sends only fields that are not completely masked out to the TC kernel module. Some of the hardware we work with requires all fields extracted from the packet along with the mask to be sent to the driver. Hardware may optimize tables based on entire fields of packet and mask of

Re: [ovs-dev] [PATCH] ovs-dpctl-top: Skip "eth()" element

2020-06-19 Thread 0-day Robot
Bleep bloop. Greetings Timothy Redaelli, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Timothy Redaelli needs to sign off. Lines checked: 36, Warnings: 0, Errors:

Re: [ovs-dev] [PATCH ovn v13 3/7] ovn-controller: Handle runtime data changes in flow output engine

2020-06-19 Thread Dumitru Ceara
On 6/19/20 1:10 PM, num...@ovn.org wrote: > From: Numan Siddique > > In order to handle runtime data changes incrementally, the flow outut > runtime data handle should know the changed runtime data. > Runtime data now tracks the changed data for any OVS interface > and SB port binding changes.

[ovs-dev] [PATCH net] openvswitch: take into account de-fragmentation in execute_check_pkt_len

2020-06-19 Thread Lorenzo Bianconi
ovs connection tracking module performs de-fragmentation on incoming fragmented traffic. Take info account if traffic has been de-fragmented in execute_check_pkt_len action otherwise we will perform the wrong nested action considering the original packet size. This issue typically occurs if

[ovs-dev] [PATCH] ovs-dpctl-top: open stdin in binary mode

2020-06-19 Thread Timothy Redaelli
On Python3 buffering can only be disabled on files opened in binary mode. --- utilities/ovs-dpctl-top.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-dpctl-top.in b/utilities/ovs-dpctl-top.in index fbe6e4f56..97dc12f27 100755 --- a/utilities/ovs-dpctl-top.in

Re: [ovs-dev] [PATCH ovn v12 5/7] tests: Enhance ovn-performance testing by adding gw router port.

2020-06-19 Thread Numan Siddique
On Fri, Jun 19, 2020 at 2:08 PM Dumitru Ceara wrote: > On 6/11/20 2:44 PM, num...@ovn.org wrote: > > From: Numan Siddique > > > > This covers the scenario of setting up/deleting of BFD tunnels > > for HA. > > > > Tested-by: Dumitru Ceara > > Signed-off-by: Numan Siddique > > Hi Numan, > >

[ovs-dev] [PATCH ovn v13 7/7] binding.c: Reorder out params of some of the static functions.

2020-06-19 Thread numans
From: Numan Siddique Acked-by: Han Zhou Signed-off-by: Numan Siddique --- controller/binding.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/controller/binding.c b/controller/binding.c index 9dbbb34df..b2c388146 100644 ---

[ovs-dev] [PATCH ovn v13 6/7] Add an util function get_unique_lport_key() for generating unique lport key.

2020-06-19 Thread numans
From: Numan Siddique Suggested-by: Dumitru Ceara Acked-by: Han Zhou Signed-off-by: Numan Siddique --- controller/binding.c | 8 controller/lflow.c | 8 lib/ovn-util.h | 8 3 files changed, 16 insertions(+), 8 deletions(-) diff --git a/controller/binding.c

[ovs-dev] [PATCH ovn v13 5/7] tests: Enhance ovn-performance testing by adding gw router port.

2020-06-19 Thread numans
From: Numan Siddique This covers the scenario of setting up/deleting of BFD tunnels for HA. Acked-by: Han Zhou Signed-off-by: Numan Siddique --- tests/ovn-performance.at | 106 +++ 1 file changed, 106 insertions(+) diff --git a/tests/ovn-performance.at

[ovs-dev] [PATCH ovn v13 4/7] ovn-controller: Use the tracked runtime data changes for flow calculation.

2020-06-19 Thread numans
From: Venkata Anil This patch processes the logical flows of tracked datapaths and tracked logical ports. To handle the tracked logical port changes, reference of logical flows to port bindings is maintained. Below are the results of some testing done with ovn-fake-multinode setup comparing

[ovs-dev] [PATCH ovn v13 1/7] I-P engine: Provide the option for an engine to clear tracked engine data in every run.

2020-06-19 Thread numans
From: Numan Siddique A new function is added in the engine node called - clear_tracked_data() to clear any engine data which was tracked during the engine run. This tracked data has to be part of the engine 'data'. engine_init_run() calls clear_tracked_data() and each engine node interested in

[ovs-dev] [PATCH ovn v13 2/7] ovn-controller: I-P for ct zone and OVS interface changes in flow output stage.

2020-06-19 Thread numans
From: Numan Siddique This patch handles ct zone changes and OVS interface changes incrementally in the flow output stage. Any changes to ct zone can be handled by running physical_run() instead of running flow_output_run(). And any changes to OVS interfaces can be either handled incrementally

[ovs-dev] [PATCH ovn v13 3/7] ovn-controller: Handle runtime data changes in flow output engine

2020-06-19 Thread numans
From: Numan Siddique In order to handle runtime data changes incrementally, the flow outut runtime data handle should know the changed runtime data. Runtime data now tracks the changed data for any OVS interface and SB port binding changes. The tracked data contains a hmap of tracked datapaths

[ovs-dev] [PATCH ovn v13 0/7] Incremental processing improvements.

2020-06-19 Thread numans
From: Numan Siddique This patch series handles port binding, datapath binding, ovs interface changes, runtime data changes, sb chassis changes incrementally. v12 -> v13 - * Addressed the review comments from Dumitru. * Fixed the test case added in p5 which was failing intermittently. *

[ovs-dev] [PATCH] ovs-dpctl-top: Skip "eth()" element

2020-06-19 Thread Timothy Redaelli
With commit efde188622ae ("odp-util: Print eth() for Ethernet flows if packet_type is absent.") "eth()" is printed for Ethernet flows if packet_type is absent, but this broke "ovs-dpctl-top" since it expects that every element has a value. This commit skips the parsing of the empty "eth()"

Re: [ovs-dev] [PATCH ovn v12 5/7] tests: Enhance ovn-performance testing by adding gw router port.

2020-06-19 Thread Dumitru Ceara
On 6/11/20 2:44 PM, num...@ovn.org wrote: > From: Numan Siddique > > This covers the scenario of setting up/deleting of BFD tunnels > for HA. > > Tested-by: Dumitru Ceara > Signed-off-by: Numan Siddique Hi Numan, This test now fails for me: 0 0 0 0 0 ovn-nbctl

Re: [ovs-dev] [PATCH ovn v12 4/7] ovn-controller: Use the tracked runtime data changes for flow calculation.

2020-06-19 Thread Dumitru Ceara
On 6/11/20 2:44 PM, num...@ovn.org wrote: > From: Venkata Anil > > This patch processes the logical flows of tracked datapaths > and tracked logical ports. To handle the tracked logical port > changes, reference of logical flows to port bindings is maintained. > > Co-Authored-by: Numan Siddique

Re: [ovs-dev] [PATCH ovn v12 2/7] ovn-controller: I-P for ct zone and OVS interface changes in flow output stage.

2020-06-19 Thread Dumitru Ceara
On 6/19/20 10:18 AM, Numan Siddique wrote: > > > On Fri, Jun 19, 2020 at 1:39 PM Numan Siddique > wrote: > > > > On Fri, Jun 19, 2020 at 1:11 PM Dumitru Ceara > wrote: > > On 6/11/20 2:43 PM, num...@ovn.org

Re: [ovs-dev] [PATCH ovn v12 2/7] ovn-controller: I-P for ct zone and OVS interface changes in flow output stage.

2020-06-19 Thread Numan Siddique
On Fri, Jun 19, 2020 at 1:39 PM Numan Siddique wrote: > > > On Fri, Jun 19, 2020 at 1:11 PM Dumitru Ceara wrote: > >> On 6/11/20 2:43 PM, num...@ovn.org wrote: >> > From: Numan Siddique >> > >> > This patch handles ct zone changes and OVS interface changes >> incrementally >> > in the flow

Re: [ovs-dev] [PATCH ovn v12 2/7] ovn-controller: I-P for ct zone and OVS interface changes in flow output stage.

2020-06-19 Thread Numan Siddique
On Fri, Jun 19, 2020 at 1:11 PM Dumitru Ceara wrote: > On 6/11/20 2:43 PM, num...@ovn.org wrote: > > From: Numan Siddique > > > > This patch handles ct zone changes and OVS interface changes > incrementally > > in the flow output stage. > > > > Any changes to ct zone can be handled by running

Re: [ovs-dev] [PATCH ovn v12 3/7] ovn-controller: Handle runtime data changes in flow output engine

2020-06-19 Thread Dumitru Ceara
On 6/11/20 2:44 PM, num...@ovn.org wrote: > From: Numan Siddique > > In order to handle runtime data changes incrementally, the flow outut > runtime data handle should know the changed runtime data. > Runtime data now tracks the changed data for any OVS interface > and SB port binding changes.

Re: [ovs-dev] [PATCH v6 ovn] Add support for DHCP domain search option (119)

2020-06-19 Thread 0-day Robot
Bleep bloop. Greetings Ankur Sharma, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or

Re: [ovs-dev] [PATCH ovn v12 2/7] ovn-controller: I-P for ct zone and OVS interface changes in flow output stage.

2020-06-19 Thread Dumitru Ceara
On 6/11/20 2:43 PM, num...@ovn.org wrote: > From: Numan Siddique > > This patch handles ct zone changes and OVS interface changes incrementally > in the flow output stage. > > Any changes to ct zone can be handled by running physical_run() instead of > running > flow_output_run(). And any

[ovs-dev] [PATCH v6 ovn] Add support for DHCP domain search option (119)

2020-06-19 Thread Ankur Sharma
From: Dhathri Purohith Domain search list is encoded according to the specifications in RFC 1035, section 4.1.4. Signed-off-by: Dhathri Purohith Signed-off-by: Ankur Sharma --- lib/actions.c | 90 - lib/ovn-l7.h| 3 ++

Re: [ovs-dev] [PATCH] netdev-offload-tc: Revert tunnel src/dst port masks handling

2020-06-19 Thread Simon Horman
On Thu, Jun 18, 2020 at 07:36:52PM +0300, Roi Dayan wrote: > > > On 2020-06-16 6:48 PM, Simon Horman wrote: > > On Tue, Jun 16, 2020 at 05:06:49PM +0300, Roi Dayan wrote: > >> > >> > >> On 2020-06-16 4:03 PM, Roi Dayan wrote: > >>> The cited commit intended to add tc support for masking tunnel

[ovs-dev] [PATCH ovn v3] Support packet metadata marking for logical router policies.

2020-06-19 Thread numans
From: Numan Siddique This patch adds a new column 'options' of type smap in the Logical_Router_Policy table in the NB DB and supports the key 'pkt_mark'. CMS can set a desired value for this key in the 'options' column. When this router policy is applied, the packet metadata is marked with the