Re: [ovs-dev] [PATCH dpdk-latest v1] build: Fix build with Sparse for rte_mbuf.h.

2020-09-03 Thread Pai G, Sunil
> -Original Message- > From: David Marchand > Sent: Thursday, September 3, 2020 1:11 PM > To: Pai G, Sunil > Cc: ovs dev ; Ilya Maximets ; > Stokes, Ian > Subject: Re: [ovs-dev] [PATCH dpdk-latest v1] build: Fix build with Sparse for > rte_mbuf.h. > > Thanks for fixing. > > Afaics in

[ovs-dev] Claves para una implementación efectiva

2020-09-03 Thread Nom 035 - Nueva Fecha.
Buenos día Quise aprovechar la oportunidad de hacerte una invitación para tomar nuestro: Nombre: NOM 035. Claves para una implementación efectiva. Fecha: Martes 18 de Septiembre Horario: 10:00 am a 13:00 pm Formato: En línea con interacción en vivo. Lugar: En Vivo desde su computadora

Re: [ovs-dev] [PATCH] ovsdb-idl.at: Wait all servers to join the cluster.

2020-09-03 Thread Ilya Maximets
On 9/4/20 12:05 AM, Flavio Leitner wrote: > On Thu, Sep 03, 2020 at 11:20:56PM +0200, Ilya Maximets wrote: >> On 6/11/20 1:45 AM, Flavio Leitner wrote: >>> The test 'Check Python IDL reconnects to leader - Python3 >>> (leader only)' fails sometimes when the first ovsdb-server >>> gets killed

Re: [ovs-dev] [PATCH] ovsdb-idl.at: Wait all servers to join the cluster.

2020-09-03 Thread Flavio Leitner
On Thu, Sep 03, 2020 at 11:20:56PM +0200, Ilya Maximets wrote: > On 6/11/20 1:45 AM, Flavio Leitner wrote: > > The test 'Check Python IDL reconnects to leader - Python3 > > (leader only)' fails sometimes when the first ovsdb-server > > gets killed before the others had joined the cluster. > > > >

[ovs-dev] Mrs.Rosa Gomes ;

2020-09-03 Thread Rosa Gomes
Dear Beloved Life is gradually passing away from me as a result of my present medical condition and my personal doctor confided in me yesterday that I have only but few more weeks to live. In view of this setback, I want to donate my estate for humanitarian assistance, since this has always

Re: [ovs-dev] [PATCH ovn 6/9] ofctrl.c: Maintain references between installed flows and desired flows.

2020-09-03 Thread Han Zhou
On Thu, Sep 3, 2020 at 12:47 PM Mark Michelson wrote: > > On 8/21/20 3:16 PM, Han Zhou wrote: > > Currently there is no link maintained between installed flows and desired > > flows. This patch maintains the mapping between them, which will be useful > > for a future patch that incrementally

[ovs-dev] Congratulation: You're a winner!

2020-09-03 Thread Euro-Millions
CONGRATULATION!!! Reference to the 1,320th Euro-Millions draw that took place on Friday 15th May 2020 at 21:00 CEST (20:00 BST) through the Euro-Millions draw system were your E-mail address emerged among lucky winners with the drawn winning numbers: Lucky numbers 11-23-32-34-39 Star Number

Re: [ovs-dev] [PATCH] ovsdb-idl: Add function to set next_remote in jsonrpc session.

2020-09-03 Thread winson wang
Hi Ilya, Thanks for your comments. Please see my reply inline. On 9/3/20 2:01 PM, Ilya Maximets wrote: External email: Use caution opening links or attachments On 8/13/20 10:52 PM, Zhen Wang wrote: OVN SouthBound DB in clustered mode, when one raft node down and online. All the

Re: [ovs-dev] [PATCH ovn 4/9] ovn-controller: Fix conjunction handling with incremental processing.

2020-09-03 Thread Han Zhou
The previous email was blocked by ML because of the embedded picture, so I replaced it with a link and resend. On Thu, Sep 3, 2020 at 2:17 PM Han Zhou wrote: > > > > On Thu, Sep 3, 2020 at 6:00 AM Mark Michelson wrote: >> >> On 9/2/20 2:55 PM, Han Zhou wrote: >> > >> > >> > On Wed, Sep 2, 2020

Re: [ovs-dev] [PATCH] ovsdb-idl.at: Wait all servers to join the cluster.

2020-09-03 Thread Ilya Maximets
On 6/11/20 1:45 AM, Flavio Leitner wrote: > The test 'Check Python IDL reconnects to leader - Python3 > (leader only)' fails sometimes when the first ovsdb-server > gets killed before the others had joined the cluster. > > Fix the function ovsdb_cluster_start_idltest to wait them > to join the

Re: [ovs-dev] [PATCH 23/23] Documentation: Update faq and NEWS for kernel 5.9

2020-09-03 Thread Ilya Maximets
On 8/21/20 12:49 AM, Greg Rose wrote: > Update the NEWS and faq now that we will support up to Linux kernel > 5.9. > > Signed-off-by: Greg Rose > --- > Documentation/faq/releases.rst | 2 +- > NEWS | 2 ++ > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff

Re: [ovs-dev] [PATCH 01/23] acinclude: Enable builds up to Linux 5.9

2020-09-03 Thread Ilya Maximets
On 8/21/20 12:49 AM, Greg Rose wrote: > Allow building openvswitch against Linux kernels up to and including > version 5.9. > > Signed-off-by: Greg Rose > > squash This looks like some rebase artifact. Also, this patch, I think, should go at the end of the series before the documentation

Re: [ovs-dev] [PATCH] ovsdb-idl: Add function to set next_remote in jsonrpc session.

2020-09-03 Thread Ilya Maximets
On 8/13/20 10:52 PM, Zhen Wang wrote: > OVN SouthBound DB in clustered mode, when one raft node down and > online. All the ovn-controller clients will not migirate back which > cause RAFT DB clients unbalanced state. > > This function provides a way to set the IDL jsonrpc session next_remote. >

Re: [ovs-dev] [PATCH ovn 6/9] ofctrl.c: Maintain references between installed flows and desired flows.

2020-09-03 Thread Mark Michelson
On 8/21/20 3:16 PM, Han Zhou wrote: Currently there is no link maintained between installed flows and desired flows. This patch maintains the mapping between them, which will be useful for a future patch that incrementally processing the flow installation without having to do the full comparison

Re: [ovs-dev] [PATCH v4 ovn] Add VXLAN support for non-VTEP datapath bindings

2020-09-03 Thread Numan Siddique
Hi Ihar, Overall the patch LGTM. I have few comments. Please see below. Thanks Numan On Wed, Aug 26, 2020 at 10:45 PM Ihar Hrachyshka wrote: > Because of limited space in VXLAN VNI to pass over all three of - > datapath id, ingress port, egress port - the implementation ignores > ingress;

Re: [ovs-dev] [PATCH] selinux: Add missing permissions for ovs-kmod-ctl

2020-09-03 Thread Ansis
Acked-by: Ansis Atteka On Thu, Sep 3, 2020, 10:03 AM Yi-Hung Wei wrote: > On RHEL 8, a SELinux policy is missing when ovs-kmod-ctl use modprobe > to load kernel modules. This patch adds the missing permissions based > on /var/log/audit/audit.log > > Example log of the AVC violations: >

Re: [ovs-dev] [OVN] ovn-northd inactivity probe interval

2020-09-03 Thread Numan Siddique
On Thu, Sep 3, 2020 at 11:19 PM Tony Liu wrote: > Hi Numan, > > I am using this thread to ask about another inactivity probe interval. > > 2020-09-03T17:24:54.908Z|68178|reconnect|ERR|tcp:10.6.20.25:6644: no > response to inactivity probe after 5 seconds, disconnecting >

Re: [ovs-dev] [OVN] ovn-northd inactivity probe interval

2020-09-03 Thread Tony Liu
Hi Numan, I am using this thread to ask about another inactivity probe interval. 2020-09-03T17:24:54.908Z|68178|reconnect|ERR|tcp:10.6.20.25:6644: no response to inactivity probe after 5 seconds, disconnecting Is that 6644 RAFT port? Is there probe on that port

Re: [ovs-dev] [PATCH v2] Remove manpages.mk from git

2020-09-03 Thread Flavio Leitner
On Thu, Sep 03, 2020 at 02:12:54PM -0300, Flavio Leitner wrote: > On Thu, Sep 03, 2020 at 04:55:43PM +0200, Timothy Redaelli wrote: > > manpages.mk is generated at build-time using sodepends.py and so there is no > > need to keep it in git. > > > > Signed-off-by: Timothy Redaelli > > --- > > v2:

Re: [ovs-dev] [OVN] ovn-northd inactivity probe interval

2020-09-03 Thread Numan Siddique
On Thu, Sep 3, 2020 at 10:36 PM Tony Liu wrote: > Hi, > > I tried to set northd_probe_interval="3" and > northd-probe-interval="3" in NB_Global options. > Still got this from ovn-northd log. > > 2020-09-03T17:00:16.445Z|22664|reconnect|ERR|tcp:10.6.20.25:6642: no

Re: [ovs-dev] [PATCH] selinux: Add missing permissions for ovs-kmod-ctl

2020-09-03 Thread Gregory Rose
On 9/3/2020 10:02 AM, Yi-Hung Wei wrote: On RHEL 8, a SELinux policy is missing when ovs-kmod-ctl use modprobe to load kernel modules. This patch adds the missing permissions based on /var/log/audit/audit.log Example log of the AVC violations: type=AVC msg=audit(1599075387.136:65): avc:

Re: [ovs-dev] [PATCH v2] Remove manpages.mk from git

2020-09-03 Thread Flavio Leitner
On Thu, Sep 03, 2020 at 04:55:43PM +0200, Timothy Redaelli wrote: > manpages.mk is generated at build-time using sodepends.py and so there is no > need to keep it in git. > > Signed-off-by: Timothy Redaelli > --- > v2: > generate manpages.mk in builddir instead of srcdir and add it to >

[ovs-dev] [OVN] ovn-northd inactivity probe interval

2020-09-03 Thread Tony Liu
Hi, I tried to set northd_probe_interval="3" and northd-probe-interval="3" in NB_Global options. Still got this from ovn-northd log. 2020-09-03T17:00:16.445Z|22664|reconnect|ERR|tcp:10.6.20.25:6642: no response to inactivity probe after 5 seconds, disconnecting

[ovs-dev] [PATCH] selinux: Add missing permissions for ovs-kmod-ctl

2020-09-03 Thread Yi-Hung Wei
On RHEL 8, a SELinux policy is missing when ovs-kmod-ctl use modprobe to load kernel modules. This patch adds the missing permissions based on /var/log/audit/audit.log Example log of the AVC violations: type=AVC msg=audit(1599075387.136:65): avc: denied { read } for pid=1472

Re: [ovs-dev] [PATCH v4 2/3] dpif-netlink: distribute polling to discreet handlers

2020-09-03 Thread Mark Gray
On 02/09/2020 00:22, Flavio Leitner wrote: > > Hi Mark, > > Thanks for following up with this patch. > See my comments below. Thanks for the review Flavio. > > On Fri, Aug 28, 2020 at 06:03:04PM +0100, Mark Gray wrote: >> From: Aaron Conole >> >> Currently, the channel handlers are polled

[ovs-dev] [PATCH ovn] northd: fix empty_lb_backends controller_event for IPv6

2020-09-03 Thread Lorenzo Bianconi
Introduce missing square brackets defining IPv6 empty_lb_backends controller_event logical flows in order to properly extract vip ip and port from the related string Fixes: bb9f2b9ce56c ("ovn-northd: Consider load balancer active backends in router pipeline") Fixes: 821e1e54abcb ("OVN: use

[ovs-dev] [PATCH v2 ovn 1/2] chassis: Fix the way encaps are updated for a chassis record.

2020-09-03 Thread Dumitru Ceara
ovn-controller always stores the last configured system-id/chassis-id in memory regardless if the connection to the SB is up or down. This is OK as long as the change can be committed successfully when the SB DB connection comes back up. Without this change, if the chassis-id changes while the SB

[ovs-dev] [PATCH v2 ovn 2/2] chassis: Fix chassis_private record updates when the system-id changes.

2020-09-03 Thread Dumitru Ceara
Also: - Change conditional monitoring for Chassis_Private to use the chassis uuid instead of chassis name. Using the chassis->name field does not work because this is the old value of the field and would cause ovsdb-server to inform ovn-controller that the updated Chassis_Private record was

[ovs-dev] [PATCH v2] Remove manpages.mk from git

2020-09-03 Thread Timothy Redaelli
manpages.mk is generated at build-time using sodepends.py and so there is no need to keep it in git. Signed-off-by: Timothy Redaelli --- v2: generate manpages.mk in builddir instead of srcdir and add it to CLEANFILES to make "make distcleancheck" happy. Thank you Ilya and travis for

Re: [ovs-dev] [PATCH RFC 2/2] release-process: LTS transition period and policy for unmaintained branches.

2020-09-03 Thread Ilya Maximets
On 9/2/20 5:03 PM, Stokes, Ian wrote: >> While only 2 branches are formally maintained (LTS and latest release), OVS >> team usually provides stable releases for other branches too, at least for >> branches between LTS and latest. > > Thanks for proposing this Ilya, few comments inline. > >> >>

[ovs-dev] Salut

2020-09-03 Thread Valerie Bell
Désolée pour cette approche inattendue, je suis Valérie qui désire votre amitié sincère si cela ne vous a dérange pas, je reste à l écoute Sincères Salutations Valérie Bell ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH RFC 2/2] release-process: LTS transition period and policy for unmaintained branches.

2020-09-03 Thread Ilya Maximets
On 9/1/20 2:00 PM, Simon Horman wrote: > On Tue, Aug 25, 2020 at 02:24:11PM +0200, Ilya Maximets wrote: >> While only 2 branches are formally maintained (LTS and latest release), >> OVS team usually provides stable releases for other branches too, at >> least for branches between LTS and latest.

Re: [ovs-dev] [PATCH RFC 2/2] release-process: LTS transition period and policy for unmaintained branches.

2020-09-03 Thread Ilya Maximets
On 8/28/20 3:55 PM, Flavio Leitner wrote: > On Tue, Aug 25, 2020 at 02:24:11PM +0200, Ilya Maximets wrote: >> While only 2 branches are formally maintained (LTS and latest release), >> OVS team usually provides stable releases for other branches too, at >> least for branches between LTS and

Re: [ovs-dev] [PATCH ovn] pinctrl: Fix incorrect warning message for multicast querier.

2020-09-03 Thread Numan Siddique
On Thu, Sep 3, 2020 at 5:52 PM Dumitru Ceara wrote: > On 9/3/20 2:07 PM, Numan Siddique wrote: > > > > > > On Thu, Aug 27, 2020 at 8:18 PM Dumitru Ceara > > wrote: > > > > It's quite common that IP Multicast is enabled only for IPv4 or only > for > > IPv6. In

Re: [ovs-dev] [PATCH ovn 1/2] chassis: Fix the way encaps are updated for a chassis record.

2020-09-03 Thread Dumitru Ceara
On 9/3/20 3:11 PM, Dumitru Ceara wrote: [...] >> >> Oops. My bad. Thanks. >> >> I tested after applying both the patches. >> >> When I run this command - ovs-vsctl set open . external_ids:system-id=ch-2 >> I see below warning message in ovn-controller log >> >> * >>

Re: [ovs-dev] [PATCH ovn 1/2] chassis: Fix the way encaps are updated for a chassis record.

2020-09-03 Thread Dumitru Ceara
On 9/3/20 1:07 PM, Numan Siddique wrote: > > > On Thu, Sep 3, 2020 at 3:55 PM Dumitru Ceara > wrote: > > On 9/3/20 11:57 AM, Numan Siddique wrote: > > > > > > On Thu, Aug 27, 2020 at 7:32 PM Dumitru Ceara > >

[ovs-dev] [PATCH ovn] ovn-ctl: Handle cluster db upgrades for run_(nb/sb)_ovsdb

2020-09-03 Thread numans
From: Numan Siddique when ovn-ctl run_(nb_sb)_ovsdb is called, the ovsdb-server is started without passing --detach and --monoitor and the process is exec'd. For cluster mode, upgrade_cluster is never called and hence the dbs are not upraded to new schema. CMS has to handle the db upgrade

Re: [ovs-dev] [PATCH ovn 4/9] ovn-controller: Fix conjunction handling with incremental processing.

2020-09-03 Thread Mark Michelson
On 9/2/20 2:55 PM, Han Zhou wrote: On Wed, Sep 2, 2020 at 8:53 AM Mark Michelson > wrote: > > I think this can be simplified some. > > Specifically, I don't think every ovn_flow needs to have a corresponding > sb_flow_ref created. The only time that

Re: [ovs-dev] [PATCH ovs resend v2] dpctl-netdev: Add the option "pmd" for dump-flows

2020-09-03 Thread Ilya Maximets
On 8/24/20 4:44 AM, xiangxia.m@gmail.com wrote: > From: Tonghao Zhang > > "ovs-appctl dpctl/dump-flows" added the option > "pmd" which allow user to dump pmd specified. > > That option is useful to dump rules of pmd > when we have a large number of rules in dp. Is this a performance

Re: [ovs-dev] [PATCH ovs resend v2] dpctl-netdev: Add the option "pmd" for dump-flows

2020-09-03 Thread Flavio Leitner
Hi, Thanks for the patch. I just did a quick look and found couple things to discuss. On Mon, Aug 24, 2020 at 10:44:08AM +0800, xiangxia.m@gmail.com wrote: > From: Tonghao Zhang > > "ovs-appctl dpctl/dump-flows" added the option > "pmd" which allow user to dump pmd specified. > > That

Re: [ovs-dev] 答复: 答复: 答复: [PATCH] userspace: fix bad UDP performance issue of?veth

2020-09-03 Thread Ilya Maximets
On 9/3/20 4:06 AM, Yi Yang (杨燚)-云服务集团 wrote: > As I have replied per Aaron's concern, users need to use > /proc/sys/net/core/rmem_max and /proc/sys/net/core/wmem_max to set values > they prefer, final values are smaller one between > rmem_max(SOL_RCVBUF)/wmem_max(SOL_SNDBUF) and 1073741823. So

Re: [ovs-dev] Big cleanup of OVS patchwork.

2020-09-03 Thread Flavio Leitner
On Tue, Sep 01, 2020 at 04:46:56PM +0200, Ilya Maximets wrote: > For a last couple of weeks I worked on a script to update statuses of all the > patches in openvswitch patchwork instance [1], i.e. to mark all the patches > that > already applied or superseded. [...] > But anyway, OVS patchwork

Re: [ovs-dev] [PATCH ovn] ovn-nbctl: Deal with nb_cfg overflows.

2020-09-03 Thread Numan Siddique
On Fri, Aug 28, 2020 at 5:42 PM Dumitru Ceara wrote: > Reported-at: https://bugzilla.redhat.com/id=1873455 > Reported-by: Ying Xu > Signed-off-by: Dumitru Ceara > Thanks. I applied this patch to master and branch-20.06. Numan > --- > utilities/ovn-nbctl.c | 5 + > 1 file changed, 5

Re: [ovs-dev] [PATCH ovn 1/2] chassis: Fix the way encaps are updated for a chassis record.

2020-09-03 Thread Numan Siddique
On Thu, Sep 3, 2020 at 3:55 PM Dumitru Ceara wrote: > On 9/3/20 11:57 AM, Numan Siddique wrote: > > > > > > On Thu, Aug 27, 2020 at 7:32 PM Dumitru Ceara > > wrote: > > > > ovn-controller always stores the last configured > system-id/chassis-id in > > memory

Re: [ovs-dev] [PATCH ovn 1/2] chassis: Fix the way encaps are updated for a chassis record.

2020-09-03 Thread Dumitru Ceara
On 9/3/20 11:57 AM, Numan Siddique wrote: > > > On Thu, Aug 27, 2020 at 7:32 PM Dumitru Ceara > wrote: > > ovn-controller always stores the last configured system-id/chassis-id in > memory regardless if the connection to the SB is up or down. This is OK >

[ovs-dev] [PATCH v2] ovsdb: raft: add some debugging information to cluster/status command

2020-09-03 Thread Lorenzo Bianconi
Introduce the following info useful for cluster debugging to cluster/status command: - time elapsed from last start/complete election - election trigger (e.g. timeout) - number of disconnections - time elapsed from last raft messaged received Signed-off-by: Lorenzo Bianconi ---

Re: [ovs-dev] [PATCH ovn 1/2] chassis: Fix the way encaps are updated for a chassis record.

2020-09-03 Thread Numan Siddique
On Thu, Aug 27, 2020 at 7:32 PM Dumitru Ceara wrote: > ovn-controller always stores the last configured system-id/chassis-id in > memory regardless if the connection to the SB is up or down. This is OK > as long as the change can be committed successfully when the SB DB > connection comes back

Re: [ovs-dev] [PATCH ovn] Allow to run multiple controllers on the same machine

2020-09-03 Thread Dumitru Ceara
On 9/1/20 11:53 PM, Ihar Hrachyshka wrote: > On Fri, Aug 7, 2020 at 8:25 AM Dumitru Ceara wrote: >> I get the following errors when compiling with your patch: >> >> controller/ovn-controller.c:87:12: error: symbol 'controller_chassis' >> was not declared. Should it be static? >>

Re: [ovs-dev] [PATCH ovn] system-ovn.at: Make ECMP test case more resilient.

2020-09-03 Thread Dumitru Ceara
On 9/3/20 10:46 AM, Numan Siddique wrote: > > > On Wed, Sep 2, 2020 at 5:38 PM Mark Michelson > wrote: > > Thanks for the fix, Dumitru. > > Acked-by: Mark Michelson > > > > Thanks Dumitru and Mark. I applied this patch to

Re: [ovs-dev] [PATCH ovn] system-ovn.at: Make ECMP test case more resilient.

2020-09-03 Thread Numan Siddique
On Wed, Sep 2, 2020 at 5:38 PM Mark Michelson wrote: > Thanks for the fix, Dumitru. > > Acked-by: Mark Michelson > > Thanks Dumitru and Mark. I applied this patch to master. Numan > On 9/2/20 3:56 AM, Dumitru Ceara wrote: > > It's not easy to predict how OVS datapath flows exactly look so

Re: [ovs-dev] [PATCH dpdk-latest v1] build: Fix build with Sparse for rte_mbuf.h.

2020-09-03 Thread David Marchand
Thanks for fixing. Afaics in OVS, patches touching sparse headers have a sparse: prefix. + my personal preference is not to mention actual filenames in a title. How about the following title? sparse: Fix build with DPDK 20.08. On Wed, Sep 2, 2020 at 3:56 PM Sunil Pai G wrote: > > Introduction