On Thu, Jun 22, 2023 at 1:12 AM Dumitru Ceara wrote:
>
> On 6/21/23 16:50, Han Zhou wrote:
> > On Wed, Jun 21, 2023 at 2:48 AM Dumitru Ceara wrote:
> >>
> >> We don't need to explicitly add port bindings that were already bound
> >> locally. We implicitly get those because we monitor the
On Fri, Jun 23, 2023 at 11:20 AM Ilya Maximets wrote:
>
> On 6/23/23 17:15, Terry Wilson wrote:
> > Just checking in on this. I see a single test failure
> > https://github.com/ovsrobot/ovs/actions/runs/5272193494 'linux clang
> > test asan' but it looks like something related to bfd, so
This patch is same as MR: https://github.com/openvswitch/ovs/pull/415
This is Patch detail
>From ddff9ac664083e6cbed97c214935b77f25dd1228 Mon Sep 17 00:00:00 2001
From: simon
Date: Tue, 27 Jun 2023 11:20:51 +0800
Subject: [PATCH V3] Bugfix of ovs-tcpdump.
This bug will cause megaflow
On 6/23/23 14:12, Dumitru Ceara wrote:
> As far as I can tell they're used mostly for CI job definitions and
> these tend to result in long lines.
Why not just wrap them? AFAIK, syntax in most CI systems allows
line wrapping.
Not a strong opinion though, just curious.
Best regards, Ilya
On Tue, Jun 27, 2023 at 6:51 AM Dumitru Ceara wrote:
>
> On 6/8/23 07:55, Han Zhou wrote:
> > On Wed, Jun 7, 2023 at 12:39 PM Numan Siddique wrote:
> >>
> >> On Fri, Jun 2, 2023 at 12:13 AM Han Zhou wrote:
> >>>
> >>> This patch introduces a change handler for NB logical_switch within
the
> >>>
On 6/19/23 11:30, Eelco Chaudron wrote:
> Allow the ovn-system tests to run on the OVS-DPDK infrastructure,
> i.e., DPDK ports and mbuf memory.
>
> Co-authored-by: David Marchand
> Signed-off-by: David Marchand
> Signed-off-by: Eelco Chaudron
> ---
Hi Eelco,
Acked-by: Dumitru Ceara
Thanks,
On 6/19/23 11:32, Eelco Chaudron wrote:
> This patch includes changes made earlier by David in the
> ovs branch to cache the dpdk builds.
>
> Co-authored-by: David Marchand
> Signed-off-by: David Marchand
> Signed-off-by: Eelco Chaudron
> ---
>
Acked-by: Dumitru Ceara
Thanks,
Dumitru
Ilya Maximets writes:
> On 6/27/23 14:36, 0-day Robot wrote:
>> Bleep bloop. Greetings Ilya Maximets, 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.
>>
>>
>> git-am:
>> error:
On 6/26/23 19:46, David Marchand wrote:
> On Mon, Jun 26, 2023 at 4:05 PM David Marchand
> wrote:
>>
>> On Mon, Jun 26, 2023 at 3:19 PM Ilya Maximets wrote:
- if the UFO feature is "restored" in the master branch, OVS can't
expose CSUM if the guest negotiated UFO.
>>>
>>> Yep. IIUC,
On 6/27/23 15:53, Eelco Chaudron wrote:
>
>
> On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
>
>> Signed-off-by: Ilya Maximets
>
> Changes look good.
>
> Acked-by: Eelco Chaudron
>
Thanks! Applied and tagged.
Will prepare an announcement and the website update.
Best regards, Ilya
On 6/27/23 16:37, Ilya Maximets wrote:
> On 6/27/23 15:52, Eelco Chaudron wrote:
>>
>>
>> On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
>>
>>> Signed-off-by: Ilya Maximets
>>
>> Acked-by: Eelco Chaudron
>>
>
> Thanks! Applied and tagged.
> Will prepare an announcement and the website update.
On 6/27/23 15:52, Eelco Chaudron wrote:
>
>
> On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
>
>> Signed-off-by: Ilya Maximets
>
> Acked-by: Eelco Chaudron
>
Thanks! Applied and tagged.
Will prepare an announcement and the website update.
Best regards, Ilya Maximets.
On 6/27/23 15:52, Eelco Chaudron wrote:
>
>
> On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
>
>> Signed-off-by: Ilya Maximets
>> ---
>> NEWS | 3 +++
>> configure.ac | 2 +-
>> debian/changelog | 6 ++
>> 3 files changed, 10 insertions(+), 1 deletion(-)
>>
>> diff --git
On 2 Jun 2023, at 16:13, Adrian Moreno wrote:
> Currently, the netdev's speed is being calculated by taking the link's
> feature bits (using netdev_get_features()) and transforming them into
> bps.
>
> This mechanism can be both inaccurate and difficult to maintain, mainly
> because we
Since on CentOS/RHEL the builds are based on stable branches and not on
tags for debugging purpose it's better to have the downstream version as
version so it's easier to know which commits are included in a build.
This commit adds --with-version-suffix as ./configure option in
order to set an
On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
> Signed-off-by: Ilya Maximets
Changes look good.
Acked-by: Eelco Chaudron
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
> Signed-off-by: Ilya Maximets
Changes look good.
Acked-by: Eelco Chaudron
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
> Signed-off-by: Ilya Maximets
Acked-by: Eelco Chaudron
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On 27 Jun 2023, at 14:16, Ilya Maximets wrote:
> 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 9c0f9c159..673f22720 100644
Change
On 6/8/23 07:55, Han Zhou wrote:
> On Wed, Jun 7, 2023 at 12:39 PM Numan Siddique wrote:
>>
>> On Fri, Jun 2, 2023 at 12:13 AM Han Zhou wrote:
>>>
>>> This patch introduces a change handler for NB logical_switch within the
>>> 'northd' node. It specifically handles cases where logical switch
On 6/27/23 14:36, 0-day Robot wrote:
> Bleep bloop. Greetings Ilya Maximets, 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.
>
>
> git-am:
> error: Failed to merge in the changes.
>
Bleep bloop. Greetings Ilya Maximets, 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.
Patch skipped due to previous failure.
Please check this out. If you feel there has been an error,
Bleep bloop. Greetings Ilya Maximets, 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.
git-am:
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the
Bleep bloop. Greetings Ilya Maximets, 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.
Patch skipped due to previous failure.
Please check this out. If you feel there has been an error,
Bleep bloop. Greetings Ilya Maximets, 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.
git-am:
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the
On 6/26/23 16:48, Aaron Conole wrote:
Flavio Leitner writes:
Today the exit code refers to the execution of the change
in the database. However, when not using parameter --no-wait
(default), the ovs-vsctl also checks if OVSDB transactions
are successfully recorded and reload by ovs-vswitchd.
Signed-off-by: Ilya Maximets
---
NEWS | 3 ++-
debian/changelog | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index e59f9530b..0ece92a35 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,6 @@
-v3.1.2 - xx xxx
+v3.1.2 - 27 Jun 2023
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 0ece92a35..8fbf7219d 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,6 @@
+v3.1.3 - xx xxx
It's been more than 2 months since the last stable release and we
have accumulated quite a few important bug fixes, so it's time to
make another round of stable releases.
Ilya Maximets (2):
Set release date for 3.1.2.
Prepare for 3.1.3.
NEWS | 6 +-
configure.ac | 2 +-
It's been more than 2 months since the last stable release and we
have accumulated quite a few important bug fixes, so it's time to
make another round of stable releases.
Ilya Maximets (2):
Set release date for 2.17.7.
Prepare for 2.17.8.
NEWS | 6 +-
configure.ac | 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 9c0f9c159..673f22720 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,6 @@
+v2.17.8 - xx xxx
Signed-off-by: Ilya Maximets
---
NEWS | 3 ++-
debian/changelog | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index 095b0574f..9c0f9c159 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,6 @@
-v2.17.7 - xx xxx
+v2.17.7 - 27 Jun 2023
On 25/06/2023 7:04, Gavin Li wrote:
>
> On 6/23/2023 7:06 PM, Simon Horman wrote:
>> External email: Use caution opening links or attachments
>>
>>
>> On Fri, Jun 23, 2023 at 01:01:38PM +0200, Eelco Chaudron wrote:
>>>
>>> On 23 Jun 2023, at 12:57, Simon Horman wrote:
>>>
On Fri, Jun 23,
From: Gavin Li
Add a vxlan gbp offload test case:
vxlan offloads with gbp extention - ping between two ports - offloads
enabled ok
Signed-off-by: Gavin Li
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
---
tests/system-offloads-traffic.at | 50
1 file
From: Gavin Li
Kernels that do not support vxlan gbp would treat the rule that has vxlan
gbp encap action or vxlan gbp id match differently, either reject it or
just skip the action/match and continue processing the knowing ones.
To solve the issue, probe and disallow inserting rules with vxlan
From: Gavin Li
Add TC offload support for vxlan encap with gbp option
Signed-off-by: Gavin Li
Reviewed-by: Gavi Teitz
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
Acked-by: Eelco Chaudron
---
acinclude.m4 | 7
include/linux/tc_act/tc_tunnel_key.h | 17
From: Gavin Li
Most of the data members of struct tc_action{ } are defined as anonymous
struct in place. Instead of passing all members of an anonymous struct,
which is not flexible to new members being added, expose encap as named
struct and pass it entirely.
Signed-off-by: Gavin Li
From: Gavin Li
Add TC offload support for filtering vxlan tunnels with gbp option
Signed-off-by: Gavin Li
Reviewed-by: Gavi Teitz
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
Acked-by: Eelco Chaudron
---
include/linux/pkt_cls.h | 13 ++
lib/netdev-offload-tc.c | 17
From: Gavin Li
Extract vxlan gbp option decoding to odp_decode_gbp_raw to be used in
following commits.
Signed-off-by: Gavin Li
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
Acked-by: Eelco Chaudron
---
lib/odp-util.c | 9 +++--
lib/odp-util.h | 8
2 files changed, 11
From: Gavin Li
Linux kernel netlink module added NLA_F_NESTED flag checking for nested
netlink messages in 5.2. A nested message without the flag set will be
treated as malformated one. The check is optional and is controlled by
message policy. To avoid this, add NLA_F_NESTED explicitly for all
From: Gavin Li
Tc flower tunnel key options were encoded in nl_msg_put_flower_tunnel_opts
and decoded in nl_parse_flower_tunnel_opts. Only geneve was supported.
To avoid adding more arguments to the function to support more vxlan
options in the future, change the function arguments to pass
From: Gavin Li
Extract vxlan gbp option encoding to odp_encode_gbp_raw to be used in
following commits.
Signed-off-by: Gavin Li
Reviewed-by: Roi Dayan
Reviewed-by: Simon Horman
Acked-by: Eelco Chaudron
---
lib/odp-util.c | 5 +++--
lib/odp-util.h | 5 +
2 files changed, 8
Hi,
This series adds TC offload support for filtering vxlan tunnels with gbp option.
First 4 patches do some refactoring and the later patches adds the feature.
Thanks,
Roi
changelog
v6:
- fix nits in patch #7.
- fix test in patch #9 to run test on supported kernel.
v5:
- Fix byte order issue
Build the upstream Open vSwitch deb package with AF_XDP datapath
enabled when required dependencies are available.
Signed-off-by: Frode Nordahl
---
Documentation/intro/install/debian.rst | 13
debian/automake.mk | 45 ++
debian/control.in
When sb is read-only, the port claim is delayed until sb is rw.
However, before this patch, this resulted in the chassis always
claiming the port as main (while it was maybe an additional chassis).
Fixes: a7c7d4519e50 ("controller: avoid recomputes triggered by SBDB
Port_Binding updates.")
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 8
1 file changed, 8 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index f16cc2814..b7969696a 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -23636,6 +23636,10 @@ check ovs-vsctl -- add-port br-int hv2-vif2 -- \
grep -q exit immediately with zero status if any match is found.
Hence, when executing "ovs-ofctl dump-flows br-int | grep -q meter_id=1",
ovs-ofctl might fail (with error "write to stdout failed") due to the broken
pipe.
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 6 +++---
1 file
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index 83033c987..8907e4a53 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -29505,6 +29505,8 @@ for i in $(seq 1 $n); do
ovn_attach n1 br-phys 192.168.0.$i
done
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index b7969696a..c19605d64 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -4507,6 +4507,8 @@ wait_row_count MAC_Binding 2 ip="192.168.1.200"
This test could fail for two reasons:
- on slow systems, the duration of the fight between the hv might
be longer than expected, resulting in more claims than expected
- if-status module does not implement this 'fight protection'
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 2 +-
1 file
Test was failing due to two potential reasons:
- pinctrl thread was not yet started when packets were generated
- using sleep and hoping we wait long enough for packet to be received
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 23 ---
1 file changed, 8 insertions(+),
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index 9f6ba940b..83033c987 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -9079,7 +9079,7 @@ ovn-nbctl lsp-set-options lrp0-rp router-port=lrp0
ovs-ofctl monitor runs by default for OVS_CTL_TIMEOUT i.e. 30 seconds
This might not be enough to cover the duration of the test on loaded systems.
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/ovn.at
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index b9a1cd65a..ca0069132 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -13827,9 +13827,13 @@ wait_for_ports_up
check ovn-nbctl
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index 4b454a8c1..535d5f666 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -16318,7 +16318,7 @@ construct_expected_ra() {
local
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index 535d5f666..aed150563 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -36076,6 +36076,9 @@ rm -f ${OVN_SYSCONFDIR}/system-id-override
check ovn-appctl -t
Signed-off-by: Xavier Simonart
---
tests/ovn.at | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index ca0069132..1dc09a168 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -25065,13 +25065,14 @@ OVS_WAIT_FOR_OUTPUT(
Xavier Simonart (13):
tests: fixed "policy-based routing" and "route tables IPv6 --
overlapping subnets"
tests: fixed "dhcpv6 : 1 HV, 2 LS, 5 LSPs" and "external logical port"
tests: fixed "send gratuitous ARP for NAT rules on HA distributed
router"
tests: fixed "Load balancer
Following tests were randomly failing as injecting packets too early
i.e. before proper flows were installed (ports not up).
- policy-based routing: 1 HVs, 2 LSs, 1 lport/LS, 1 LR
- policy-based routing IPv6: 1 HVs, 3 LSs, 1 lport/LS, 1 LR
- route tables IPv6 -- overlapping subnets
On Mon, Jun 26, 2023 at 10:34 PM Han Zhou wrote:
>
> On Mon, Jun 26, 2023 at 7:25 AM Numan Siddique wrote:
> >
> > On Sun, Jun 18, 2023 at 11:48 AM Han Zhou wrote:
> > >
> > > For incremental processing, it is important to maintain relationship
> > > between the inputs and the logical flows
This bug is fixed in https://github.com/openvswitch/ovs/pull/415
Simon Jones
Simon Jones 于2023年6月15日周四 21:51写道:
> Add
> : ovs-disc...@openvswitch.org
>
> Simon Jones
>
>
> Simon Jones 于2023年6月15日周四 21:48写道:
>
>> I found only vxlan topology has this bug.
>>
>> Other topology such as
61 matches
Mail list logo