Re: [ovs-dev] OVN LTS/non-LTS bug fix backport strategy (was: Re: [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.)

On Mon, Apr 17, 2023 at 12:01 PM Ilya Maximets wrote: > > On 4/17/23 19:20, Han Zhou wrote: > > > > > > On Mon, Apr 17, 2023 at 9:24 AM Numan Siddique > wrote: > >> > >> On Mon, Apr 17, 2023 at 5:57 AM Dumitru Ceara mailto:dce...@redhat.com>> wrote: > >> > > >> > On 4/17/23 11:55, Dumitru Ceara

Re: [ovs-dev] [PATCH ovn] Expose distributed gateway port information in NB DB

(Sorry that my previous reply includes redundant texts. Please ignore that one and use this version :D) On Mon, Apr 17, 2023 at 7:18 AM Lucas Martins wrote: > > Thanks all for the discussion and all the ideas here. > > After reading the emails, I think it boils down to two proposed approaches: >

Re: [ovs-dev] [PATCH ovn] Expose distributed gateway port information in NB DB

On Mon, Apr 17, 2023 at 7:18 AM Lucas Martins wrote: > > Thanks all for the discussion and all the ideas here. > > After reading the emails, I think it boils down to two proposed approaches: > > 1) CMS to continue to connect to the Southbound database if they need > information about the physical

Re: [ovs-dev] OVN LTS/non-LTS bug fix backport strategy (was: Re: [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.)

On 4/17/23 19:20, Han Zhou wrote: > > > On Mon, Apr 17, 2023 at 9:24 AM Numan Siddique > wrote: >> >> On Mon, Apr 17, 2023 at 5:57 AM Dumitru Ceara > > wrote: >> > >> > On 4/17/23 11:55, Dumitru Ceara wrote: >> > > On 4/14/23 18:01, Han Zhou

Re: [ovs-dev] OVN LTS/non-LTS bug fix backport strategy (was: Re: [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.)

On Mon, Apr 17, 2023 at 9:24 AM Numan Siddique wrote: > > On Mon, Apr 17, 2023 at 5:57 AM Dumitru Ceara wrote: > > > > On 4/17/23 11:55, Dumitru Ceara wrote: > > > On 4/14/23 18:01, Han Zhou wrote: > > >> > > >> > > >> On Fri, Apr 14, 2023 at 2:16 AM Dumitru Ceara > >>

[ovs-dev] [PATCH] system-offloads-traffic: Remove tc ingress pps check on meter offload.

Caught during some code review. The incriminated commit had put an unneeded check on tc ingress support for the meter offloading test. Note: SUPPORT_TC_INGRESS_PPS had been reworked in the commit 5f0fdf5e2c2e ("test: Move check for tc ingress pps support to test script."). Fixes: 5660b89a309d

Re: [ovs-dev] OVN LTS/non-LTS bug fix backport strategy (was: Re: [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.)

On Mon, Apr 17, 2023 at 5:57 AM Dumitru Ceara wrote: > > On 4/17/23 11:55, Dumitru Ceara wrote: > > On 4/14/23 18:01, Han Zhou wrote: > >> > >> > >> On Fri, Apr 14, 2023 at 2:16 AM Dumitru Ceara >> > wrote: > >>> > >>> On 4/14/23 04:15, Han Zhou wrote: > On Thu,

Re: [ovs-dev] [PATCH v10] utilities/ofctl: add-meters for save and restore

On Tue, Apr 11, 2023 at 09:25:43PM +0800, Wan Junjie wrote: > put dump-meters' result in one line so add-meters can handle. > save and restore meters when restart ovs. > bundle functions are not implemented in this patch. > > Signed-off-by: Wan Junjie > > --- > v10: > merge oneline to verbosity

Re: [ovs-dev] [PATCH ovn] ci: Add job to build Fedora RPMs on every push/PR

This is better than nothing, copr would be the usually thing to have, but looks like it needs a lot of administrative duties, we can improve this in the future. +1 On Wed, Apr 12, 2023 at 10:59 AM Ales Musil wrote: > In order to have up-to-date Fedora RPMs from main > branch add job that will

[ovs-dev] [PATCH ovn 3/3] ci: Switch Cirrus CI to use the new image

Use the image with preinstalled dependencies also on Cirrus CI. Signed-off-by: Ales Musil --- .cirrus.yml | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/.cirrus.yml b/.cirrus.yml index 10869d8a2..bd4cd08aa 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -1,7

[ovs-dev] [PATCH ovn 2/3] ci: Use container to run the tests

Move the ci.sh script into .ci folder and remove the linux-prepare.sh as it is no longer needed with all the requirements installed in container. Signed-off-by: Ales Musil --- {utilities/containers => .ci}/ci.sh | 0 .ci/linux-prepare.sh| 21 ---

[ovs-dev] [PATCH ovn 1/3] ci: Conditionally install gcc-multilib

The gcc-multilib is needed for x86 build on x86_64, but the package is not available for arm64. We need to conditonally install only when we run the x86 job. Signed-off-by: Ales Musil --- .ci/linux-build.sh | 7 +++ 1 file changed, 7 insertions(+) diff --git a/.ci/linux-build.sh

Re: [ovs-dev] [PATCH ovn] Expose distributed gateway port information in NB DB

Thanks all for the discussion and all the ideas here. After reading the emails, I think it boils down to two proposed approaches: 1) CMS to continue to connect to the Southbound database if they need information about the physical location of the resources. That would avoid the inefficiency of

[ovs-dev] [PATCH v10] netdev-dpdk: add control plane protection support

Some control protocols are used to maintain link status between forwarding engines (e.g. LACP). When the system is not sized properly, the PMD threads may not be able to process all incoming traffic from the configured Rx queues. When a signaling packet of such protocols is dropped, it can cause

Re: [ovs-dev] [PATCH v4] util: fix an issue that thread name cannot be set

On 17 Apr 2023, at 8:54, Songtao Zhan wrote: > To: d...@openvswitch.org, > i.maxim...@ovn.org > > The name of the current thread consists of a name with a maximum > length of 16 bytes and a thread ID. The final name may be longer > than 16 bytes. If the name is longer than 16 bytes, the

Re: [ovs-dev] OVN LTS/non-LTS bug fix backport strategy (was: Re: [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.)

On 4/17/23 11:55, Dumitru Ceara wrote: > On 4/14/23 18:01, Han Zhou wrote: >> >> >> On Fri, Apr 14, 2023 at 2:16 AM Dumitru Ceara > > wrote: >>> >>> On 4/14/23 04:15, Han Zhou wrote: On Thu, Apr 13, 2023 at 12:44 PM Han Zhou > > wrote: >

[ovs-dev] OVN LTS/non-LTS bug fix backport strategy (was: Re: [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.)

On 4/14/23 18:01, Han Zhou wrote: > > > On Fri, Apr 14, 2023 at 2:16 AM Dumitru Ceara > wrote: >> >> On 4/14/23 04:15, Han Zhou wrote: >> > On Thu, Apr 13, 2023 at 12:44 PM Han Zhou > wrote: >> >> >> >> >> >> >> >> On Thu, Apr 13, 2023 at 12:35 

[ovs-dev] [Patch ovn] pinctrl: fix restart of controller when bfd min_tx set to 1

when create bfd entry, set min_tx to 1, (tx_timeout * 25)/100 is zero, which cause controller to restart! Signed-off-by: wangchuanlei --- controller/pinctrl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/controller/pinctrl.c b/controller/pinctrl.c index 795847729..7817da6e0 100644 ---

Re: [ovs-dev] [PATCH net-next 0/3] selftests: openvswitch: add support for testing upcall interface

Hello: This series was applied to netdev/net-next.git (main) by David S. Miller : On Fri, 14 Apr 2023 09:17:47 -0400 you wrote: > The existing selftest suite for openvswitch will work for regression > testing the datapath feature bits, but won't test things like adding > interfaces, or the

[ovs-dev] [PATCH v4] util: fix an issue that thread name cannot be set

To: d...@openvswitch.org, i.maxim...@ovn.org The name of the current thread consists of a name with a maximum length of 16 bytes and a thread ID. The final name may be longer than 16 bytes. If the name is longer than 16 bytes, the thread name will fail to be set Signed-off-by: Songtao Zhan