Re: [ovs-dev] [PATCH ovn] controller: fix ip buffering with static routes

2020-05-22 Thread Lorenzo Bianconi
> Hi Lorenzo, Hi Ankur, > > Good catch. > > a. If not already considered, then i think it is a candidate for 20.06 > b. Need not be done in this patch, but it will be good to have a testcase > which validates the draining of buffered packets. We already have some tests in system-ovn.at and in

Re: [ovs-dev] [PATCH ovn v7 2/9] ovn-controller: I-P for SB port binding and OVS interface in runtime_data.

2020-05-22 Thread Numan Siddique
On Fri, May 22, 2020 at 1:17 AM Han Zhou wrote: > On Thu, May 21, 2020 at 3:49 AM Numan Siddique wrote: > > > > > > > > On Thu, May 21, 2020 at 6:57 AM Han Zhou wrote: > >> > >> Hi Numan, please see my comments below. > > > > > > Thanks for the review. Please see below for a few comments. > > >

[ovs-dev] [PATCH] ofproto-dpif-xlate: ovs-tcpdump cannot capture incomming vxlan packets

2020-05-22 Thread hepeng.0320
when running ovs-tcpdump -i ethX and the port is used as the incomming port for a vxlan port. The callstack for the upcall: mirror_ingress_packet mirror_packet output_normal compose_output_action compose_output_action__ terminate_native_tunnel will xlate the action into a tnl_pop action, not an

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: ovs-tcpdump cannot capture incomming vxlan packets

2020-05-22 Thread 贺鹏
ignore this patch, forget to sign-off the patch. hepeng.0320 于2020年5月22日周五 下午5:02写道: > > when running ovs-tcpdump -i ethX and the port is used as the incomming port > for a vxlan port. > > The callstack for the upcall: > > mirror_ingress_packet > mirror_packet > output_normal > compose_output_ac

[ovs-dev] [PATCH] ofproto-dpif-xlate: ovs-tcpdump cannot capture incomming vxlan packets

2020-05-22 Thread hepeng.0320
when running ovs-tcpdump -i ethX and the port is used as the incomming port for a vxlan port. The callstack for the upcall: mirror_ingress_packet mirror_packet output_normal compose_output_action compose_output_action__ terminate_native_tunnel will xlate the action into a tnl_pop action, not an

[ovs-dev] [PATCH v13 1/1] Avoid dp_hash recirculation for balance-tcp bond selection mode

2020-05-22 Thread Vishal Deep Ajmera via dev
Problem: In OVS-DPDK, flows with output over a bond interface of type “balance-tcp” (using a hash on TCP/UDP 5-tuple) get translated by the ofproto layer into "HASH" and "RECIRC" datapath actions. After recirculation, the packet is forwarded to the bond member port based on 8-bits of the d

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: ovs-tcpdump cannot capture incomming vxlan packets

2020-05-22 Thread 0-day Robot
Bleep bloop. Greetings hepeng.0320, 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 hepeng.0320 needs to sign off. WARNING: Line is 83 characters long (recommended l

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: ovs-tcpdump cannot capture incomming vxlan packets

2020-05-22 Thread 0-day Robot
Bleep bloop. Greetings hepeng.0320, 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 hepeng.0320 needs to sign off. WARNING: Unexpected sign-offs from developers who

[ovs-dev] [PATCH v13 0/1] Balance-tcp bond mode optimization

2020-05-22 Thread Vishal Deep Ajmera via dev
v12->v13: Addressed comments from Ilya, Eelco and Matteo (offline review). Fixed issues with the patch found by Matteo during testing. Rebased to OVS master. v11->v12: Addressed most of comments from Ilya and Eelco. https://mail.openvswitch.org/pipermail/ovs-dev/2020-February/367832.html htt

Re: [ovs-dev] [PATCH v13 1/1] Avoid dp_hash recirculation for balance-tcp bond selection mode

2020-05-22 Thread 0-day Robot
Bleep bloop. Greetings Vishal Deep Ajmera, 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: Comment with 'xxx' marker #557 FILE: lib/dpif-netdev.c:7349: /* XXX: net

Re: [ovs-dev] [PATCH] ovs-ctl: Don't set hostname as external-id

2020-05-22 Thread Daniel Alvarez Sanchez
On Fri, May 22, 2020 at 2:17 AM Han Zhou wrote: > > > > On Thu, May 21, 2020 at 3:19 AM Daniel Alvarez Sanchez > wrote: > > > > On Thu, May 21, 2020 at 9:22 AM Han Zhou wrote: > > > > > > > > > > > > On Wed, May 20, 2020 at 8:52 AM Daniel Alvarez > > > wrote: > > > > > > > > ovs-ctl started t

[ovs-dev] Multi Million Dollar Project

2020-05-22 Thread FINANCE M
Please permit me to write to you irrespective of the fact we have not met before. However, your reputable profile gives me the impression to engage you to be a suitable partner in a multi million dollar project which I will give further details upon receipt of your response to this mail. My nam

[ovs-dev] (no subject)

2020-05-22 Thread Gabriele Cerami
___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] (no subject)

2020-05-22 Thread Gabriele Cerami
___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 2/2] Introduce async IO in JSONRPC

2020-05-22 Thread Mark Michelson
Hi Anton, One general note about this submission is that I see some coding guidelines violations. One is when it comes to definitions of functions. The guidelines say to write these as return_type func_name(params) { } But some of your functions are written as: return_type func_name(params)

[ovs-dev] Lineamientos técnicos de seguridad sanitaria

2020-05-22 Thread Stps - Estrategias generales de control
Buenos día Quise aprovechar la oportunidad de hacerte una invitación para tomar nuestro curso: Nombre: Lineamientos técnicos de seguridad sanitaria en el entorno laboral. Horario: de 3:00 a 6:00 Hrs. ¿Cuándo?: Jueves 28 de Mayo Formato: En línea con interacción en vivo. Lugar: En Vivo desde su

[ovs-dev] [PATCH ovn] Documentation: add a quick start for prospective contributors

2020-05-22 Thread Gabriele Cerami
This change adds a small guide that may be useful to prospective contributors that start from scratch. Signed-off-by: Gabriele Cerami --- Documentation/internals/contributing/index.rst | 1 + .../internals/contributing/quickstart.rst | 202 + 2 files changed, 2

Re: [ovs-dev] [PATCH ovn] Documentation: add a quick start for prospective contributors

2020-05-22 Thread 0-day Robot
Bleep bloop. Greetings Gabriele Cerami, 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: Line is 90 characters long (recommended limit is 79) #84 FILE: Documentation/intern

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

2020-05-22 Thread Numan Siddique
On Thu, May 21, 2020 at 12:02 PM Han Zhou wrote: > On Wed, May 20, 2020 at 12:41 PM 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 phy

[ovs-dev] [PATCH] ovsdb: Add raft memory usage to memory report.

2020-05-22 Thread Ilya Maximets
Memory reports could be found in logs or by calling 'memory/show' appctl command. For ovsdb-server it includes information about db cells, monitor connections with their backlog size, etc. But it doesn't contain any information about memory consumed by raft. Backlogs of raft connections could be

[ovs-dev] [PATCH v6] AB bonding: Add "primary" interface concept

2020-05-22 Thread Jeff Squyres via dev
In AB bonding, if the current active slave becomes disabled, a replacement slave is arbitrarily picked from the remaining set of enabled slaves. This commit adds the concept of a "primary" slave: an interface that will always be (or become) the current active slave if it is enabled. The rationale

Re: [ovs-dev] [PATCH ovn] controller: fix ip buffering with static routes

2020-05-22 Thread Ankur Sharma
Hi Lorenzo, Please find my replies inline. Regards, Ankur From: Lorenzo Bianconi Sent: Friday, May 22, 2020 12:48 AM To: Ankur Sharma Cc: ovs-dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH ovn] controller: fix ip buffering with static routes > Hi Lorenzo,

Re: [ovs-dev] [PATCH ovn] ovn: introduce IP_SRC_POLICY stage in ingress router pipeline

2020-05-22 Thread Han Zhou
On Wed, May 20, 2020 at 12:43 PM Lorenzo Bianconi < lorenzo.bianc...@redhat.com> wrote: > > > On Wed, May 20, 2020 at 7:38 AM Dumitru Ceara wrote: > > > > > > On 5/19/20 6:42 PM, Lorenzo Bianconi wrote: > > > > In order to fix the issues introduced by commit > > > > c0bf32d72f8b ("Manage ARP proce

Re: [ovs-dev] [PATCH] ovsdb: Add raft memory usage to memory report.

2020-05-22 Thread Han Zhou
On Fri, May 22, 2020 at 10:36 AM Ilya Maximets wrote: > > Memory reports could be found in logs or by calling 'memory/show' > appctl command. For ovsdb-server it includes information about db > cells, monitor connections with their backlog size, etc. But it > doesn't contain any information abou

[ovs-dev] [PATCH v2 ovn] ovn: introduce IP_SRC_POLICY stage in ingress router pipeline

2020-05-22 Thread Lorenzo Bianconi
In order to fix the issues introduced by commit c0bf32d72f8b ("Manage ARP process locally in a DVR scenario "), restore previous configuration of table 9 in ingress router pipeline and introduce a new stage called 'ip_src_policy' used to set the src address info in order to not distribute FIP traff

[ovs-dev] [PATCH v7] AB bonding: Add "primary" interface concept

2020-05-22 Thread Jeff Squyres via dev
In AB bonding, if the current active slave becomes disabled, a replacement slave is arbitrarily picked from the remaining set of enabled slaves. This commit adds the concept of a "primary" slave: an interface that will always be (or become) the current active slave if it is enabled. The rationale

Re: [ovs-dev] [PATCH v2 ovn] ovn: introduce IP_SRC_POLICY stage in ingress router pipeline

2020-05-22 Thread Han Zhou
Hi Lorenzo, Sorry that I replied to v1 right before you send v2. Please check my comments there. Thanks, Han On Fri, May 22, 2020 at 1:50 PM Lorenzo Bianconi < lorenzo.bianc...@redhat.com> wrote: > In order to fix the issues introduced by commit > c0bf32d72f8b ("Manage ARP process locally in a

Re: [ovs-dev] [PATCH ovn] ovn: introduce IP_SRC_POLICY stage in ingress router pipeline

2020-05-22 Thread Lorenzo Bianconi
> On Wed, May 20, 2020 at 12:43 PM Lorenzo Bianconi < > lorenzo.bianc...@redhat.com> wrote: > > [...] > > > > Regards, > > Lorenzo > > > > Hi Lorenzo, > Hi Han, thx for the detailed explanation > Thanks for explaining the scenario (here and also today's OVN meeting). > Going through the patc

Re: [ovs-dev] [PATCH] ovs-ctl: Don't set hostname as external-id

2020-05-22 Thread Terry Wilson
On Wed, May 20, 2020 at 10:52 AM Daniel Alvarez wrote: > ovs-ctl started to add the hostname as external-id [0] at some point. > > However, this can be problematic as if it's already set by an external > entity it will get overwritten. In RHEL systems, systemd will invoke > ovs-ctl to start OVS a

Re: [ovs-dev] [PATCH ovn v7 2/9] ovn-controller: I-P for SB port binding and OVS interface in runtime_data.

2020-05-22 Thread Han Zhou
On Fri, May 22, 2020 at 1:51 AM Numan Siddique wrote: > > > > On Fri, May 22, 2020 at 1:17 AM Han Zhou wrote: >> >> On Thu, May 21, 2020 at 3:49 AM Numan Siddique wrote: >> > >> > >> > >> > On Thu, May 21, 2020 at 6:57 AM Han Zhou wrote: >> >> >> >> Hi Numan, please see my comments below. >> >

Re: [ovs-dev] [PATCH ovn v7 2/9] ovn-controller: I-P for SB port binding and OVS interface in runtime_data.

2020-05-22 Thread Numan Siddique
On Sat, May 23, 2020 at 3:10 AM Han Zhou wrote: > On Fri, May 22, 2020 at 1:51 AM Numan Siddique wrote: > > > > > > > > On Fri, May 22, 2020 at 1:17 AM Han Zhou wrote: > >> > >> On Thu, May 21, 2020 at 3:49 AM Numan Siddique wrote: > >> > > >> > > >> > > >> > On Thu, May 21, 2020 at 6:57 AM Ha

Re: [ovs-dev] [PATCH ovn v7 2/9] ovn-controller: I-P for SB port binding and OVS interface in runtime_data.

2020-05-22 Thread Han Zhou
On Fri, May 22, 2020 at 3:07 PM Numan Siddique wrote: > > > > On Sat, May 23, 2020 at 3:10 AM Han Zhou wrote: >> >> On Fri, May 22, 2020 at 1:51 AM Numan Siddique wrote: >> > >> > >> > >> > On Fri, May 22, 2020 at 1:17 AM Han Zhou wrote: >> >> >> >> On Thu, May 21, 2020 at 3:49 AM Numan Siddiqu

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

2020-05-22 Thread Han Zhou
On Fri, May 22, 2020 at 10:29 AM Numan Siddique wrote: > > > > On Thu, May 21, 2020 at 12:02 PM Han Zhou wrote: >> >> On Wed, May 20, 2020 at 12:41 PM wrote: >> > >> > From: Numan Siddique >> > >> > This patch handles ct zone changes and OVS interface changes incrementally >> > in the flow outp

[ovs-dev] Попередження щодо облікового запису

2020-05-22 Thread POSTMASTER via dev
Шановний користувач ID Ваші вхідні повідомлення були опубліковані в очікуванні через наше останнє оновлення. Будь ласка, дотримуйтеся наведеної нижче інформаційної посилання, щоб підтвердити та оновити свою поштову скриньку та збільшити послугу квот на поштову скриньку. Натисніть тут, щоб он