Re: [ovs-dev] [PATCH branch-3.3 0/2] Release patches for v3.3.1.

On 6/7/24 16:42, Kevin Traynor wrote: > On 07/06/2024 15:01, Ilya Maximets wrote: >> We didn't make a stable release for a while. It's definitely time >> to make one. >> >> Ilya Maximets (2): >> Set release date for 3.3.1. >> Prepare for 3.3.2. >> >> NEWS | 6 +- >> configure.

Re: [ovs-dev] [PATCH ovn] northd: Skip arp-proxy flows if the lsp is a router port.

On 6/5/24 14:03, Enrique Llorente Pastora wrote: > On Wed, Jun 5, 2024 at 12:12 PM Lorenzo Bianconi < > lorenzo.bianc...@redhat.com> wrote: > >>> On 5/14/24 18:44, Lorenzo Bianconi wrote: Skip proxy-arp logical flows for traffic that is entering the logical switch pipeline from a lsp of

Re: [ovs-dev] [PATCH branch-2.17 0/2] Release patches for v2.17.10.

On 07/06/2024 15:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. > > Ilya Maximets (2): > Set release date for 2.17.10. > Prepare for 2.17.11. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +

Re: [ovs-dev] [PATCH branch-3.0 0/2] Release patches for v3.0.7.

On 07/06/2024 15:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. > > Ilya Maximets (2): > Set release date for 3.0.7. > Prepare for 3.0.8. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++-

Re: [ovs-dev] [PATCH branch-3.2 0/2] Release patches for v3.2.3.

On 07/06/2024 15:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. > > Ilya Maximets (2): > Set release date for 3.2.3. > Prepare for 3.2.4. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++-

Re: [ovs-dev] [PATCH branch-3.1 0/2] Release patches for v3.1.5.

On 07/06/2024 15:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. > > Ilya Maximets (2): > Set release date for 3.1.5. > Prepare for 3.1.6. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++-

Re: [ovs-dev] [PATCH branch-3.3 0/2] Release patches for v3.3.1.

On 07/06/2024 15:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. > > Ilya Maximets (2): > Set release date for 3.3.1. > Prepare for 3.3.2. > > NEWS | 6 +- > configure.ac | 2 +- > debian/changelog | 8 +++-

Re: [ovs-dev] [PATCH branch-3.2 0/2] Release patches for v3.2.3.

On 7 Jun 2024, at 16:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. ACK on the series, changes look good. Acked-by: Eelco Chaudron > Ilya Maximets (2): > Set release date for 3.2.3. > Prepare for 3.2.4. > > NEWS

Re: [ovs-dev] [PATCH branch-3.3 0/2] Release patches for v3.3.1.

On 7 Jun 2024, at 16:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. ACK on the series, changes look good. Acked-by: Eelco Chaudron > Ilya Maximets (2): > Set release date for 3.3.1. > Prepare for 3.3.2. > > NEWS

Re: [ovs-dev] [PATCH branch-3.1 0/2] Release patches for v3.1.5.

On 7 Jun 2024, at 16:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. ACK on the series, changes look good. Acked-by: Eelco Chaudron > Ilya Maximets (2): > Set release date for 3.1.5. > Prepare for 3.1.6. > > NEWS

Re: [ovs-dev] [PATCH branch-3.0 0/2] Release patches for v3.0.7.

On 7 Jun 2024, at 16:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. ACK on the series, changes look good. Acked-by: Eelco Chaudron > Ilya Maximets (2): > Set release date for 3.0.7. > Prepare for 3.0.8. > > NEWS

Re: [ovs-dev] [PATCH branch-2.17 0/2] Release patches for v2.17.10.

On 7 Jun 2024, at 16:01, Ilya Maximets wrote: > We didn't make a stable release for a while. It's definitely time > to make one. ACK on the series, changes look good. Acked-by: Eelco Chaudron > Ilya Maximets (2): > Set release date for 2.17.10. > Prepare for 2.17.11. > > NEWS

Re: [ovs-dev] [PATCH v2] checkpatch: Extend and move extra_keywords list to file.

On 7 Jun 2024, at 15:46, Mike Pattrick wrote: > On Fri, Jun 7, 2024 at 2:35 AM Eelco Chaudron wrote: >> >> >> >> On 6 Jun 2024, at 3:07, Mike Pattrick wrote: >> >>> This patch extends the extra_keywords list from 324 to 747 keywords and >>> moves this list to a separate file. The methodology us

Re: [ovs-dev] [PATCH ovn v5 1/2] northd: Make `vxlan_mode` a global variable.

I’ve posted new version (v6) of patch set: https://patchwork.ozlabs.org/project/ovn/list/?series=410010 > On 6 Jun 2024, at 22:40, Ihar Hrachyshka wrote: > > On Thu, Jun 6, 2024 at 1:27 AM Vladislav Odintsov > wrote: > >> Thanks Mark for such a detailed answer! >> >

[ovs-dev] [PATCH branch-3.3 1/2] Set release date for 3.3.1.

Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 31d235a40..aaf8d4b4a 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v3.3.1 - xx xxx +v3.3.1 - 07 Jun 2024

[ovs-dev] [PATCH branch-3.3 2/2] Prepare for 3.3.2.

Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index aaf8d4b4a..f62d06ffb 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v3.3.2 - xx xxx +--

[ovs-dev] [PATCH branch-3.3 0/2] Release patches for v3.3.1.

We didn't make a stable release for a while. It's definitely time to make one. Ilya Maximets (2): Set release date for 3.3.1. Prepare for 3.3.2. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.45.0

[ovs-dev] [PATCH branch-3.2 2/2] Prepare for 3.2.4.

Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 43ab6d806..492b096db 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v3.2.4 - xx xxx +--

[ovs-dev] [PATCH branch-3.2 1/2] Set release date for 3.2.3.

Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 8baf3d2aa..43ab6d806 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v3.2.3 - xx xxx +v3.2.3 - 07 Jun 2024

[ovs-dev] [PATCH branch-3.2 0/2] Release patches for v3.2.3.

We didn't make a stable release for a while. It's definitely time to make one. Ilya Maximets (2): Set release date for 3.2.3. Prepare for 3.2.4. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.45.0

[ovs-dev] [PATCH branch-3.1 1/2] Set release date for 3.1.5.

Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 44a3e6247..9604939b3 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v3.1.5 - xx xxx +v3.1.5 - 07 Jun 2024

[ovs-dev] [PATCH branch-3.1 2/2] Prepare for 3.1.6.

Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 9604939b3..cf0700ab3 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v3.1.6 - xx xxx +--

[ovs-dev] [PATCH branch-3.1 0/2] Release patches for v3.1.5.

We didn't make a stable release for a while. It's definitely time to make one. Ilya Maximets (2): Set release date for 3.1.5. Prepare for 3.1.6. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.45.0

[ovs-dev] [PATCH branch-3.0 0/2] Release patches for v3.0.7.

We didn't make a stable release for a while. It's definitely time to make one. Ilya Maximets (2): Set release date for 3.0.7. Prepare for 3.0.8. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.45.0

[ovs-dev] [PATCH branch-3.0 2/2] Prepare for 3.0.8.

Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 088bf0dd2..504c70b6b 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v3.0.8 - xx xxx +--

[ovs-dev] [PATCH branch-3.0 1/2] Set release date for 3.0.7.

Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 8550e8e89..088bf0dd2 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v3.0.7 - xx xxx +v3.0.7 - 07 Jun 2024

[ovs-dev] [PATCH branch-2.17 0/2] Release patches for v2.17.10.

We didn't make a stable release for a while. It's definitely time to make one. Ilya Maximets (2): Set release date for 2.17.10. Prepare for 2.17.11. NEWS | 6 +- configure.ac | 2 +- debian/changelog | 8 +++- 3 files changed, 13 insertions(+), 3 deletions(-) -- 2.

[ovs-dev] [PATCH branch-2.17 1/2] Set release date for 2.17.10.

Signed-off-by: Ilya Maximets --- NEWS | 3 ++- debian/changelog | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index e91072dd6..a809e8e3d 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,6 @@ -v2.17.10 - xx xxx +v2.17.10 - 07 Jun 2024 -

[ovs-dev] [PATCH branch-2.17 2/2] Prepare for 2.17.11.

Signed-off-by: Ilya Maximets --- NEWS | 3 +++ configure.ac | 2 +- debian/changelog | 6 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index a809e8e3d..8ba8a8d62 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,6 @@ +v2.17.11 - xx xxx +

[ovs-dev] [PATCH ovn v6 2/2] northd: Add support for disabling vxlan mode.

Commit [1] introduced a "VXLAN mode" concept. It brought a limitation for available tunnel IDs because of lack of space in VXLAN VNI. In VXLAN mode OVN is limited by 4095 datapaths (LRs or non-transit LSs) and 2047 logical ports per datapath. Prior to this patch VXLAN mode was enabled automatical

[ovs-dev] [PATCH ovn v6 1/2] northd: Make `vxlan_mode` a global variable.

This simplifies code and subsequent commit to explicitely disable VXLAN mode is based on these changes. Also "VXLAN mode" term is introduced in ovn-architecture man page. Signed-off-by: Vladislav Odintsov --- northd/en-global-config.c | 3 +- northd/northd.c | 76

[ovs-dev] [PATCH ovn v6 0/2] Add support to disable VXLAN mode.

v6: - Addressed Mark's review comments: 1. Removed global variable "vxlan_mode" change from "global" engine node. 2. Configuration knob "disable_vxlan_mode" was renamed to "vxlan_mode" v5: - Addressed Ihar's review comments: 1. fixed errors after incorrect conflicts solving on rebas

Re: [ovs-dev] [PATCH] python: ovs: flow: Fix nested check_pkt_len acts.

On 6/6/24 19:53, Adrián Moreno wrote: > On Thu, Jun 06, 2024 at 06:00:26PM GMT, Ilya Maximets wrote: >> On 6/6/24 17:15, Adrian Moreno wrote: >>> Add check_pkt_len action to the decoder list that it, itself, uses. >>> >>> This makes nested check_pkt_len (i.e:a check_pkt_len inside another) >>> work

Re: [ovs-dev] [PATCH] python: idl: Fix index not being updated on row modification.

On 6/6/24 20:55, Terry Wilson wrote: > On Thu, Jun 6, 2024 at 10:41 AM Dumitru Ceara wrote: >> >> On 5/27/24 23:39, Ilya Maximets wrote: >>> When a row is modified, python IDL doesn't perform any operations on >>> existing client-side indexes. This means that if the column on which >>> index is c

Re: [ovs-dev] [PATCH 2/3] python: ovsdb-idl: Make IndexedRows mirror hmap.

On 4/11/24 15:57, Terry Wilson wrote: > I tried to get this to thread under the "ovsdb-idl: potential issues > with the persistent UUID implementation" thread, but failed. This is > one potential solution to that which mirrors the current C IDL > implementation which seems to work for the most part

Re: [ovs-dev] [PATCH v2] checkpatch: Extend and move extra_keywords list to file.

On Fri, Jun 7, 2024 at 2:35 AM Eelco Chaudron wrote: > > > > On 6 Jun 2024, at 3:07, Mike Pattrick wrote: > > > This patch extends the extra_keywords list from 324 to 747 keywords and > > moves this list to a separate file. The methodology used to create this > > list was running the spell checker

Re: [ovs-dev] [External] : Re: [PATCH ovn] northd: Skip arp-proxy flows if the lsp is a router port.

On 6/7/24 15:26, brendan.do...@oracle.com wrote: > > > On 07/06/2024 14:20, Dumitru Ceara wrote: >> On 6/7/24 12:24, brendan.do...@oracle.com wrote: >>> >>> On 06/06/2024 17:34, Dumitru Ceara wrote: On 6/5/24 16:52, Brendan Doyle via dev wrote: > So I'm wondering will this break the LSP

Re: [ovs-dev] [External] : Re: [PATCH ovn] northd: Skip arp-proxy flows if the lsp is a router port.

On 07/06/2024 14:20, Dumitru Ceara wrote: On 6/7/24 12:24, brendan.do...@oracle.com wrote: On 06/06/2024 17:34, Dumitru Ceara wrote: On 6/5/24 16:52, Brendan Doyle via dev wrote: So I'm wondering will this break the LSP option: *o**p**t**i**o**n**s*  *:*  *a**r**p**_**p**r**o**x*

Re: [ovs-dev] [External] : Re: [PATCH ovn] northd: Skip arp-proxy flows if the lsp is a router port.

On 6/7/24 12:24, brendan.do...@oracle.com wrote: > > > On 06/06/2024 17:34, Dumitru Ceara wrote: >> On 6/5/24 16:52, Brendan Doyle via dev wrote: >>> >>> So I'm wondering will this break the LSP option: *o**p**t**i**o**n**s*  *:*  *a**r**p**_**p**r**o**x**y*: optional string >>

Re: [ovs-dev] [Patch] ovsdb-client: Document "--timeout" option in help.

I made a silly typo in the commit message s/infomration/information. Would it be possible to fix it when the commit is applied, or is it worth posting v2? Thanks, Martin. On Fri, 2024-06-07 at 13:53 +0200, Martin Kalcok wrote: > Add infomration about "-t" and "--timeout" options for ovsdb-client.

[ovs-dev] [Patch] ovsdb-client: Document "--timeout" option in help.

Add infomration about "-t" and "--timeout" options for ovsdb-client. The option is documented in the "Other options" section, similar to how "ovs-appctl" has it. Signed-off-by: Martin Kalcok --- ovsdb/ovsdb-client.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ovsdb/ovsdb-client.c b/ovs

Re: [ovs-dev] [PATCH] datapath-windows: Fix parsing of split buffers in OvsGetTcpHeader.

Sure, happy to help if we have a patch. Best, Wenying On Thu, Jun 6, 2024 at 7:19 PM Alin Serdean wrote: > Hi Ilya, > > Thanks for the patch. Will try to get a setup and test it. > > In theory if we have a new analyze target of a newer visual studio it > should flag those types of issues. > > W

Re: [ovs-dev] [PATCH v2] checkpatch: Don't warn on pointer to pointer.

On Fri, Jun 07, 2024 at 12:03:15PM +0100, Simon Horman wrote: > On Fri, Jun 07, 2024 at 07:06:21AM +, Adrián Moreno wrote: > > On Fri, Jun 07, 2024 at 08:57:11AM GMT, Eelco Chaudron wrote: > > > > > > > > > On 5 Jun 2024, at 15:51, Adrian Moreno wrote: > > > > > > > Current regexp used to check

Re: [ovs-dev] [PATCH v2] checkpatch: Don't warn on pointer to pointer.

On Fri, Jun 07, 2024 at 07:06:21AM +, Adrián Moreno wrote: > On Fri, Jun 07, 2024 at 08:57:11AM GMT, Eelco Chaudron wrote: > > > > > > On 5 Jun 2024, at 15:51, Adrian Moreno wrote: > > > > > Current regexp used to check whitespaces around operators does not > > > consider that there can be more

Re: [ovs-dev] [Patch v2] ovsdb-client: Add "COLUMN" arg to help for 'dump'.

On Fri, Jun 07, 2024 at 11:30:55AM +0200, Martin Kalcok wrote: > Help text for 'ovsdb-client dump' does not mention that it's capable > of dumping a specific column's contents if the user supplies the > column's name as a fourth positional argument. > > Signed-off-by: Martin Kalcok Acked-by: Sim

Re: [ovs-dev] [PATCH 2/2] ci: Restore vhost-user unit tests in check-dpdk.

On 07/06/2024 11:08, Eelco Chaudron wrote: > > > On 6 Jun 2024, at 15:11, David Marchand wrote: > >> Following a rework in the DPDK cache, the PATH variable is incorrectly >> set, resulting in dpdk-testpmd not being available. >> Because of this, vhost-user unit tests were skipped in GHA runs. >

Re: [ovs-dev] [External] : Re: [PATCH ovn] northd: Skip arp-proxy flows if the lsp is a router port.

On 06/06/2024 17:34, Dumitru Ceara wrote: On 6/5/24 16:52, Brendan Doyle via dev wrote: So I'm wondering will this break the LSP option:     *o**p**t**i**o**n**s*  *:*  *a**r**p**_**p**r**o**x**y*: optional string    Optional.  A  list  of  MAC  and  addresses/cidrs  or jus

Re: [ovs-dev] [PATCH 2/2] ci: Restore vhost-user unit tests in check-dpdk.

On 6 Jun 2024, at 15:11, David Marchand wrote: > Following a rework in the DPDK cache, the PATH variable is incorrectly > set, resulting in dpdk-testpmd not being available. > Because of this, vhost-user unit tests were skipped in GHA runs. > > Fixes: 8893e24d9d09 ("dpdk: Update to use v23.11."

Re: [ovs-dev] [PATCH 1/2] system-dpdk: Fix socket conflict when starting testpmd.

On 6 Jun 2024, at 15:11, David Marchand wrote: > The DPDK telemetry library tries to connect to existing socket files so > that it knows whether it can take over them. > > As was reported by Christian, following a fix in DPDK that got backported > in v23.11.1, vhost-user unit tests that have bo

Re: [ovs-dev] [Patch] ovsdb-client: Add "COLUMN" arg to help for 'dump'.

Thank you for correction Ilya. Sorry I originally missed your review, but the v2 is up now. Martin. On Tue, 2024-05-28 at 19:35 +0200, Ilya Maximets wrote: > On 5/28/24 19:35, Ilya Maximets wrote: > > On 5/21/24 10:38, Martin Kalcok wrote: > > > Help text for 'ovsdb-client dump' does not mention

[ovs-dev] [Patch v2] ovsdb-client: Add "COLUMN" arg to help for 'dump'.

Help text for 'ovsdb-client dump' does not mention that it's capable of dumping a specific column's contents if the user supplies the column's name as a fourth positional argument. Signed-off-by: Martin Kalcok --- ovsdb/ovsdb-client.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

Re: [ovs-dev] [PATCH 2/2] ci: Restore vhost-user unit tests in check-dpdk.

On 06/06/2024 14:11, David Marchand wrote: > Following a rework in the DPDK cache, the PATH variable is incorrectly > set, resulting in dpdk-testpmd not being available. > Because of this, vhost-user unit tests were skipped in GHA runs. > > Fixes: 8893e24d9d09 ("dpdk: Update to use v23.11.") > Sig

Re: [ovs-dev] [PATCH 1/2] system-dpdk: Fix socket conflict when starting testpmd.

On 06/06/2024 14:11, David Marchand wrote: > The DPDK telemetry library tries to connect to existing socket files so > that it knows whether it can take over them. > > As was reported by Christian, following a fix in DPDK that got backported > in v23.11.1, vhost-user unit tests that have both OVS

Re: [ovs-dev] [PATCH v2] checkpatch: Don't warn on pointer to pointer.

On Fri, Jun 07, 2024 at 08:57:11AM GMT, Eelco Chaudron wrote: > > > On 5 Jun 2024, at 15:51, Adrian Moreno wrote: > > > Current regexp used to check whitespaces around operators does not > > consider that there can be more than one "*" together to express pointer > > to pointer. > > > > As a result

Re: [ovs-dev] [PATCH] python: ovs: flow: Fix nested check_pkt_len acts.

On 6 Jun 2024, at 17:15, Adrian Moreno wrote: > Add check_pkt_len action to the decoder list that it, itself, uses. > > This makes nested check_pkt_len (i.e:a check_pkt_len inside another) > work. > > Signed-off-by: Adrian Moreno Thanks for fixing this, changes look good to me. Guess only a f