On Tue, 28 Sep 2021 12:47:27 -0700 Toms Atteka wrote:
> diff --git a/include/uapi/linux/openvswitch.h
> b/include/uapi/linux/openvswitch.h
> index a87b44cd5590..dc6eb5f6399f 100644
> --- a/include/uapi/linux/openvswitch.h
> +++ b/include/uapi/linux/openvswitch.h
> @@ -346,6 +346,13 @@ enum
This change adds a new OpenFlow field OFPXMT_OFB_IPV6_EXTHDR and
packets can be filtered using ipv6_ext flag.
Signed-off-by: Toms Atteka
---
include/uapi/linux/openvswitch.h | 12 +++
net/openvswitch/flow.c | 140 +++
net/openvswitch/flow.h | 14
On Tue, Sep 28, 2021 at 7:51 AM Nicolas Dichtel
wrote:
>
> Le 27/09/2021 à 21:12, Cpp Code a écrit :
> > To use this code there is a part of code in the userspace. We want to
> > keep compatibility when we only update userspace part code or only
> > kernel part code. This means we should have
Bleep bloop. Greetings Lorenzo Bianconi, 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 80 characters long (recommended limit is 79)
#56 FILE:
Introduce the two following commands to configure and dump static
L2 address bindings:
$ovn-sbctl set-mac-binding 192.168.1.100 00:11:22:33:44:55 vif0
$ovn-sbctl list-mac-binding
[ 192.168.1.100 00:11:22:33:44:55 S ]
https://bugzilla.redhat.com/show_bug.cgi?id=1845111
Signed-off-by: Lorenzo
Hi Timothy,
I can't claim to be a subject matter expert, but the changes make sense
on the surface. I can also confirm the OVN testsuite passes with this
change.
Acked-by: Mark Michelson
On 9/10/21 10:34 AM, Timothy Redaelli wrote:
Currently, pyOpenSSL is half-deprecated upstream and so
> -Original Message-
> From: Ilya Maximets
> Sent: Tuesday, September 21, 2021 1:45 PM
> To: Amber, Kumar ; ovs-dev@openvswitch.org
> Cc: ktray...@redhat.com; i.maxim...@ovn.org; Stokes, Ian
> ; f...@sysclose.org; echau...@redhat.com; Van Haaren,
> Harry
> Subject: Re: [PATCH v3 0/6]
New lib/plug-provider module contains the infrastructure for
registering plug provider classes which may be hosted inside or
outside the core OVN repository.
New controller/plug module adds internal interface for interacting
with the plug providers.
Extend build system to allow enabling building
Add port by interfaces index - To be able to effectively remove
ports previously plugged by us we need to look up ports by
interface records.
Add `enable-dummy-plug` option - To enable testing of the plugging
infrastructure without building OVN with an external plugging
library we include a dummy
When OVN is linked with an appropriate plugging implementation,
CMS can request OVN to plug individual lports into the local
Open vSwitch instance.
The port and instance record will be maintained during the lifetime
of the lport and it will be removed on release of lport.
Signed-off-by: Frode
Introduce infrastructure for plug providers and add feature to
ovn-controller to add and remove ports on the integration bridge as
directed by CMS through Logical_Switch_Port options.
Traditionally it has been the CMSs responsibility to create Virtual
Interfaces (VIFs) as part of instance
Currently the tests/automake.mk adds source files from the project
as needed and rebuilds these for the unit test programs.
Use the already built objects instead.
Signed-off-by: Frode Nordahl
---
tests/automake.mk | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git
The `get_tunnel_type` (and associated enum) and `get_bridge`
functions are used in modules other than the `ovn-controller`
module.
Since the `ovn-controller` module also has the `main` function
this organization makes it hard or impossible to unit test.
Signed-off-by: Frode Nordahl
---
The binding module maintains a shash of lports bound to the local
chassis. Other modules have interest in the same data for lookup,
and it would be wasteful to reimplement the same data structure
elsewhere.
The incremental processing engine already makes sharing of the
data between nodes safe
The `can_bind_on_this_chassis` helper is useful outside of the
binding module. Since it is related to lports renaming it to
`lport_can_bind_on_this_chassis` and putting it into the lport
module appears to be appropriate.
Signed-off-by: Frode Nordahl
---
controller/binding.c | 17
Make use of the common functions for maintaining OVS ports.
Signed-off-by: Frode Nordahl
---
controller/patch.c | 39 ++-
1 file changed, 6 insertions(+), 33 deletions(-)
diff --git a/controller/patch.c b/controller/patch.c
index a661025da..0d0d53894 100644
Up until now the controller patch module has been the only
consumer of functions to maintain OVS ports and interfaces.
With the introduction of infrastructure for plugging providers
these functions will also be consumed by the controller binding
module.
As such we introduce a new module called
To allow for ovn-controller to efficiently monitor all
Port_Binding records destined to it, we add a new
requested_chassis column with weakRef to the Chassis table.
The ovn-controller can monitor this column and only process
records for its chassis UUID before having claimed the port.
northd
Improve the efficiency of the requested-chassis feature by using
the new Southbound Port_Binding:requested_chassis column instead
of each chassis performing string comparison for every
Port_Binding record processed.
Signed-off-by: Frode Nordahl
---
controller/binding.c| 36
This test currently passes, but is broken in two ways.
1) The `fetch_column` helper should be used to retrieve the value
of hostname, not `fetch` wich results in a "fetch: command not
found" error which is currently not caught by the test. As a
consequence the requested-chassis option
On Mon, Sep 27, 2021 at 8:40 AM Dumitru Ceara wrote:
>
> Recent OVS commits have been optimizing the ovsdb-data module, with
> immediate goal of optimizing ovsdb-server operations. However, some of
> the ovsdb-data functions are also used on the client side (e.g.,
> db-ctl-base.c, ovsdb-idl.c).
Le 27/09/2021 à 21:12, Cpp Code a écrit :
> To use this code there is a part of code in the userspace. We want to
> keep compatibility when we only update userspace part code or only
> kernel part code. This means we should have same values for constants
> and we can only add new ones at the end
On Fri, Sep 24, 2021 at 02:30:16PM +0200, Eelco Chaudron wrote:
> Hi Martin,
>
> See my comments below...
>
> Cheers,
>
> Eelco
>
> On 30 Aug 2021, at 14:40, Martin Varghese wrote:
>
> > From: Martin Varghese
> >
> > The encap & decap actions are extended to support MPLS packet type.
> >
Bleep bloop. Greetings Frode Nordahl, 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.
build:
sphinx-build -b html -W -n -d ./Documentation/_build/doctrees ./Documentation
New lib/plug-provider module contains the infrastructure for
registering plug provider classes which may be hosted inside or
outside the core OVN repository.
New controller/plug module adds internal interface for interacting
with the plug providers.
Extend build system to allow enabling building
Introduce infrastructure for plug providers and add feature to
ovn-controller to add and remove ports on the integration bridge as
directed by CMS through Logical_Switch_Port options.
Traditionally it has been the CMSs responsibility to create Virtual
Interfaces (VIFs) as part of instance
When OVN is linked with an appropriate plugging implementation,
CMS can request OVN to plug individual lports into the local
Open vSwitch instance.
The port and instance record will be maintained during the lifetime
of the lport and it will be removed on release of lport.
Signed-off-by: Frode
Currently the tests/automake.mk adds source files from the project
as needed and rebuilds these for the unit test programs.
Use the already built objects instead.
Signed-off-by: Frode Nordahl
---
tests/automake.mk | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git
The binding module maintains a shash of lports bound to the local
chassis. Other modules have interest in the same data for lookup,
and it would be wasteful to reimplement the same data structure
elsewhere.
The incremental processing engine already makes sharing of the
data between nodes safe
Add port by interfaces index - To be able to effectively remove
ports previously plugged by us we need to look up ports by
interface records.
Add `enable-dummy-plug` option - To enable testing of the plugging
infrastructure without building OVN with an external plugging
library we include a dummy
The `get_tunnel_type` (and associated enum) and `get_bridge`
functions are used in modules other than the `ovn-controller`
module.
Since the `ovn-controller` module also has the `main` function
this organization makes it hard or impossible to unit test.
Signed-off-by: Frode Nordahl
---
Make use of the common functions for maintaining OVS ports.
Signed-off-by: Frode Nordahl
---
controller/patch.c | 39 ++-
1 file changed, 6 insertions(+), 33 deletions(-)
diff --git a/controller/patch.c b/controller/patch.c
index a661025da..0d0d53894 100644
Up until now the controller patch module has been the only
consumer of functions to maintain OVS ports and interfaces.
With the introduction of infrastructure for plugging providers
these functions will also be consumed by the controller binding
module.
As such we introduce a new module called
The `can_bind_on_this_chassis` helper is useful outside of the
binding module. Since it is related to lports renaming it to
`lport_can_bind_on_this_chassis` and putting it into the lport
module appears to be appropriate.
Signed-off-by: Frode Nordahl
---
controller/binding.c | 17
Improve the efficiency of the requested-chassis feature by using
the new Southbound Port_Binding:requested_chassis column instead
of each chassis performing string comparison for every
Port_Binding record processed.
Signed-off-by: Frode Nordahl
---
controller/binding.c| 36
To allow for ovn-controller to efficiently monitor all
Port_Binding records destined to it, we add a new
requested_chassis column with weakRef to the Chassis table.
The ovn-controller can monitor this column and only process
records for its chassis UUID before having claimed the port.
northd
This test currently passes, but is broken in two ways.
1) The `fetch_column` helper should be used to retrieve the value
of hostname, not `fetch` wich results in a "fetch: command not
found" error which is currently not caught by the test. As a
consequence the requested-chassis option
On 9/28/21 13:30, Amber, Kumar wrote:
> Hi llya,
>
> The test-case fails with the following build command on the master branch.
>
> Pass:
> ./configure --with-dpdk=static CFLAGS=""
> Fails:
> ./configure --with-dpdk=static CFLAGS="-msse4.2"
>
> Testing on ovs-master branch, running test case
Hi llya,
The test-case fails with the following build command on the master branch.
Pass:
./configure --with-dpdk=static CFLAGS=""
Fails:
./configure --with-dpdk=static CFLAGS="-msse4.2"
Testing on ovs-master branch, running test case like this
$ make check TESTSUITEFLAGS="779"
Based on the
Hi,
On Mon, Sep 27, 2021 at 5:18 PM Aaron Conole wrote:
>
> Greetings all,
>
> Around September 20th, the hardware running the "0-day Robot" that
> provides CI testing for some upstream projects suffered a catastrophic
> failure. This required a lot of internal scrambling to get replacement
>
Hi Simon,
On 9/24/2021 3:03 PM, Simon Horman wrote:
On Thu, Sep 16, 2021 at 11:38:52AM +0300, Chris Mi wrote:
Add two sFlow offload test caes:
3: sflow offloads with sampling=1 - ping between two ports - offloads
enabled ok
4: sflow offloads with sampling=2 - ping between two ports -
41 matches
Mail list logo