I sat down with this patch series and ran a case of stamping out 800
of the following patterns (each in its own openstack project) to see how
this patch set improves things within lflow_run:
n1 -- r1 -- x1
n1 is an openstack network with a dhcp subnet
r1 is an openstack router
x1 is an openstac
Sorry for the partial repeat, but my previous reply bounced off of dev and
then I realized that I hit send too soon...
That's why I said a patch set series - this patch was based on top of
another patch. Applying this patch by itself results in the compile error
you are seeing...
I'm trying to w
7;t get too confused, and run
it on another port (say 6641).
(3) Modify northd to be able to talk to both ports
(4) Modify the networking-ovn plugin to talk to port 6641.
Corrections and improvements that decrease the amount of work will be
appreciated...
Ryan Moats (regXboi)
"dev" wrot
Ben Pfaff wrote on 02/05/2016 01:01:56 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/05/2016 01:02 PM
> Subject: Re: [ovs-dev] [PATCH] [ovn-controller] [RFC] Poor man's
> lflow incremental processing
>
> On Wed, Feb
Ben Pfaff wrote on 02/05/2016 04:20:04 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/05/2016 04:20 PM
> Subject: Re: [ovs-dev] [PATCH] [ovn-controller] [RFC] Poor man's
> lflow incremental processing
>
> On Fri, Feb
at start up
Ryan Moats
IRC handle: regXboi
"dev" wrote on 02/08/2016 09:41:25 AM:
> From: Michael
> To: dev@openvswitch.org
> Date: 02/08/2016 09:41 AM
> Subject: [ovs-dev] [PATCH 1/1] ovn/utilities: Separating OVN NB and
> SB databases. Run dbs in separate
> Sent by:
"dev" wrote on 02/05/2016 05:09:44 PM:
> From: Ryan Moats/Omaha/IBM@IBMUS
> To: Ben Pfaff
> Cc: dev@openvswitch.org
> Date: 02/05/2016 05:10 PM
> Subject: Re: [ovs-dev] [PATCH] [ovn-controller] [RFC] Poor man's
> lflow incremental processing
> Sent by:
Russell Bryant wrote on 02/08/2016 10:24:29 AM:
> From: Russell Bryant
> To: Ryan Moats/Omaha/IBM@IBMUS, Michael
> Cc: dev@openvswitch.org
> Date: 02/08/2016 10:24 AM
> Subject: Re: [ovs-dev] [PATCH 1/1] ovn/utilities: Separating OVN NB
> and SBdatabases. Run dbs in sep
te on 02/08/2016 11:27:12 AM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/08/2016 11:27 AM
> Subject: Re: [ovs-dev] [PATCH] [ovn-controller] [RFC] Poor man's
> lflow incremental processing
>
> On Mon, Feb 08, 2016 at 10:34
Changed over from trying to work with logical datapaths to actually
working with logical flows. A test with deploying 400 copies of this
pattern in openstack (n1 -- r1 -- n2) results in 22083 flows in the
logical flow table with the peak number of flows being worked in a
single lflow_run pass bei
From: RYAN D. MOATS
Add incremental processing of logical flows in ovn-controller by
tracking changes to the match column of the Logical_Flow
OVN SB table. Code is included to properly handle the order of
checked sequence numbers and rechecking of rows skipped due to
a logical_datapath not exist
V3 fixes handling of deleted records passed by _FOR_EACH_TRACKED and
closes a missed continue to avoid skipping a bypassed record from a later
run.
I've tested this with a setup of 8000 openstack routers, 16000 openstack
networks and 32000 openstack ports. At full load, lflow_run now takes
about
From: RYAN D. MOATS
Add incremental processing of logical flows in ovn-controller by
tracking changes to the match column of the Logical_Flow
OVN SB table. Code is included to properly handle the order of
checked sequence numbers and rechecking of rows skipped due to
a logical_datapath not exist
Russell Bryant wrote on 02/12/2016 01:59:27 PM:
> From: Russell Bryant
> To: Ryan Moats/Omaha/IBM@IBMUS, dev@openvswitch.org
> Date: 02/12/2016 01:59 PM
> Subject: Re: [ovs-dev] [PATCH] [ovn-controller] Add logical flow
> incremental processing
>
> On 02/10/2016 05:14
Make test 1737 output the OF flows from all three hypervisors to help
in case something goes wrong.
Signed-off-by: RYAN D. MOATS
---
tests/ovn.at | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index f4117b6..e13a9c3 100644
--- a/t
From: RYAN D. MOATS
Make flow table persistent in ovn controller
This is a prerequisite for incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/ofctrl.c | 99 ---
ovn/controller/ofctrl.h |2 +
ovn/controller/ovn-con
While this patch passes the ovn test suite, it is not working correctly, so
consider this is "Self-NACK"
and I'll work on getting it fixed tomorrow...
Ryan
Ryan Moats/Omaha/IBM@IBMUS wrote on 02/16/2016 02:30:20 PM:
> From: Ryan Moats/Omaha/IBM@IBMUS
> To: dev@openv
From: RYAN D. MOATS
Add useful information to ovn E2E test
Make test 1737 output the OF flows from all three hypervisors to help
in case something goes wrong.
Signed-off-by: RYAN D. MOATS
---
tests/ovn.at | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/tes
From: RYAN D. MOATS
This code changes lflow_run to do incremental processing rather
than recreating the flow table on each run.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/lflow.c | 60 +++---
ovn/controller/lflow.h |5 ++-
ovn/contro
From: RYAN D. MOATS
Make flow table persistent in ovn controller
This is a prerequisite for incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/ofctrl.c | 118 +++
ovn/controller/ofctrl.h |2 +
ovn/controller/ovn-con
This is a prerequisite for incremental processing.
Signed-off-by: Ryan Moats
---
ovn/controller/ofctrl.c | 118 +++
ovn/controller/ofctrl.h |2 +
ovn/controller/ovn-controller.c |4 +-
3 files changed, 87 insertions(+), 37 deletions
Make test 1737 output the OF flows from all three hypervisors to help
in case something goes wrong.
Signed-off-by: Ryan Moats
---
tests/ovn.at | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index f4117b6..e13a9c3 100644
--- a
This code changes lflow_run to do incremental processing rather
than recreating the flow table on each run.
Signed-off-by: Ryan Moats
---
ovn/controller/lflow.c | 60 +++---
ovn/controller/lflow.h |5 ++-
ovn/controller/ovn-controller.c
This series of patches changes lflow_run from processing the entire
southbound logical flow table to doing incremental processing each
run.
Ryan Moats (3):
Add useful information to ovn E2E test
Make flow table persistent in ovn controller
Add incremental processing
ovn/controller/lflow.c
> If a logical port has two ipv4 addresses and one ipv6 address
> it will be stored as ["MAC IPv41 IPv42 IPv61"] instead of
> ["MAC IPv41", "MAC IPv42", "MAC IPv61"].
>
> Signed-off-by: Numan Siddique
Acked-By: Ryan Moats
_
; priority=80, match=(inport == "portname" && eth.src == 00:00:00:00:00:01
> && ip6), action=(drop;)
>
> priority=50, match=(inport == "portname" && eth.src ==
00:00:00:00:00:01),
> action=(next;)
>
> Signed-off-by: Numan Siddique
Acked-By: Ryan Moats
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
> For each lport, adds a priority 90 lflow in ls_in_port_sec and
ls_out_port_sec
> stages to allow ipv6 traffic for
> - known ipv6 addresses
> - link local address of the lport
> - ip6 packet with ip6.src = :: and
> - ip6.dst=ff00::/8
>
> Signed-off-by: Numan Siddiqu
> From: Ben Pfaff
>
> Signed-off-by: Numan Siddique
Acked-By: Ryan Moats
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Han Zhou wrote on 02/18/2016 01:34:03 AM:
[snipped for BW]
> > + /* loop through all the flows to see if there is an old flow to be
> > + * removed - do so if the old flow has the same priority,
> table, and match
> > + * but a different action or if the old flow has the same
> pri
Han Zhou wrote on 02/18/2016 12:10:09 PM:
> From: Han Zhou
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: "dev@openvswitch.org"
> Date: 02/18/2016 12:10 PM
> Subject: Re: [ovs-dev] [PATCH v6 2/3] [ovn-controller] Make flow
> table persistent in ovn controller
>
>
&
All-
Having chatted with Ben on IRC late yesterday afternoon, I sat down and
made a list of events
that would require incremental processing to restart at sequence number 0
because they might
change how a Logical Flow row gets mapped to OF flows without the Logical
Flow changing.
Looking at the
From: RYAN D. MOATS
This is a prerequisite for incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/ofctrl.c | 118 +++
ovn/controller/ofctrl.h |2 +
ovn/controller/ovn-controller.c |4 +-
3 files changed, 87 inser
From: RYAN D. MOATS
Currently changes are added to the front of the track list, so
they are looped through in LIFO order. Incremental processing
is more efficient with a FIFO presentation, so add changes to the
back of the track list.
---
lib/ovsdb-idl.c |6 +++---
1 files changed, 3 inserti
From: RYAN D. MOATS
Persist across runs so that a change to this simap can be used
as a trigger for resetting incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/lflow.c | 18 --
1 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/ovn/controlle
From: RYAN D. MOATS
Persist logical_datapaths across runs so that a change can be used
as a trigger to reset incremental flow processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/ovn-controller.c | 14 +-
1 files changed, 1 insertions(+), 13 deletions(-)
diff --git a/ovn/
From: RYAN D. MOATS
This code changes lflow_run to do incremental process of the
logical flow table rather than processing the full table each run.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c|2 +
ovn/controller/lflow.c | 53
From: RYAN D. MOATS
Make test 1737 output the OF flows from all three hypervisors to help
in case something goes wrong.
Signed-off-by: RYAN D. MOATS
---
tests/ovn.at | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index f4117b6..
From: RYAN D. MOATS
This series of patches changes lflow_run from processing the
entire southbound logical flow table to doing incremental
processing during each run.
To make this work, several pre-requisites are necessary, and
these are contained in the initial patches of the series.
RYAN D. M
=$OVN_DIR/ovnnb_db.sock
> + DB_NB_PID=$OVN_DIR/ovnnb_db.pid
> + DB_NB_FILE=$OVN_DIR/ovnnb_db.db
> + DB_NB_PORT=6641
> +
> + DB_SB_SOCK=$OVN_DIR/ovnsb_db.sock
> + DB_SB_PID=$OVN_DIR/ovnsb_db.pid
> + DB_SB_FILE=$OVN_DIR/ovnsb_db.db
> + DB_SB_PORT=6642
> +
> + DB_NB_SCHEMA=$datadir/ovn-nb.ovsschema
> + DB_SB_SCHEMA=$datadir/ovn-sb.ovsschema
> +
> + DB_SOCK=$rundir/db.sock
> + DB_CONF_FILE=$dbdir/conf.db
> +
> + OVN_NORTHD_PRIORITY=-10
> + OVN_NORTHD_WRAPPER=
> + OVN_CONTROLLER_PRIORITY=-10
> + OVN_CONTROLLER_WRAPPER=
> +
> + OVS_RUNDIR=${OVS_RUNDIR:-${rundir}}
> + OVN_RUNDIR=${OVN_RUNDIR:-${OVS_RUNDIR}}
> +
> + OVN_CONTROLLER_LOG="-vconsole:emer -vsyslog:err -vfile:info"
> + OVN_NORHD_LOG="-vconsole:emer -vsyslog:err -vfile:info"
Typo: should be OVN_NORTHD_LOG
> + OVN_OVSDB_LOG="-vconsole:off"
> }
>
> set_option () {
Ryan Moats
(regXboi)
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
> From: RYAN D. MOATS
>
> This series of patches changes lflow_run from processing the
> entire southbound logical flow table to doing incremental
> processing during each run.
>
> To make this work, several pre-requisites are necessary, and
> these are contained in the initial patches of the ser
Ben Pfaff wrote on 02/25/2016 03:11:52 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/25/2016 03:12 PM
> Subject: Re: [ovs-dev,v7,2/6,ovn-controller] Reverse change tracking
order
>
> On Fri, Feb 19, 2016 at 11:25:08AM -0
Ben Pfaff wrote on 02/25/2016 03:19:32 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/25/2016 03:20 PM
> Subject: Re: [ovs-dev,v7,3/6,ovn-controller] Make flow table persistent
>
> On Fri, Feb 19, 2016 at 11:25:09AM -0600, Ryan
Ben Pfaff wrote on 02/25/2016 03:31:56 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/25/2016 03:32 PM
> Subject: Re: [ovs-dev] [PATCH v7 4/6] [ovn-controller] Persist ports
> simap in logical_datapath
>
> On Fri, Feb 19,
Ben Pfaff wrote on 02/25/2016 03:41:16 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/25/2016 03:41 PM
> Subject: Re: [ovs-dev] [PATCH v7 6/6] [ovn-controller] Add
> incremental logical flow proessing
>
> On Fri, Feb 19,
Ben Pfaff wrote on 02/25/2016 03:49:37 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 02/25/2016 03:49 PM
> Subject: Re: [ovs-dev,v7,3/6,ovn-controller] Make flow table persistent
>
> On Thu, Feb 25, 2016 at 03:38:10PM -0600, Ryan
"discuss" wrote on 02/29/2016 08:37:02
AM:
> From: Ryan Moats/Omaha/IBM@IBMUS
> To: disc...@openvswitch.org
> Date: 02/29/2016 08:37 AM
> Subject: [ovs-discuss] OVSDB change tracking - is the delete case
> really usable?
> Sent by: "discuss"
>
> All-
Ryan Moats/Omaha/IBM wrote on 02/23/2016 12:44:53 PM:
> From: Ryan Moats/Omaha/IBM
> To: arnaldimich...@gmail.com
> Date: 02/23/2016 12:44 PM
> Subject: Fw: [ovs-dev] [PATCH V4] Separation of NB & SB dbs.
>
> FH
>
> > From V3 removed any unnecessary code ch
Michael Arnaldi wrote on 02/29/2016 09:55:54 AM:
> From: Michael Arnaldi
> To: Ryan Moats/Omaha/IBM@IBMUS
> Date: 02/29/2016 09:56 AM
> Subject: Re: [ovs-dev] [PATCH V4] Separation of NB & SB dbs.
>
> Hi Ryan,
>
> Sorry for delay it was a crazy week at work!
>
&
From: RYAN D. MOATS
Updated to ensure split of log files as well as fixed unit testing.
RYAN D. MOATS (1):
Separating OVN NB and SB database processes
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 169 +-
ovn/utilities/
From: RYAN D. MOATS
OVN NB & SB DB's should be run in separate ovsdb-server processes
and should run with ovn-ctl start_northd / stop_northd
Signed-off-by: RYAN D. MOATS
Signed-off-by: Michael Arnaldi
---
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 169 +
From: RYAN D. MOATS
Fixed typos to link the correct log variable to the correct ovsdb process
RYAN D. MOATS (1):
Separating OVN NB and SB database processes
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 169 +-
ovn/util
From: RYAN D. MOATS
OVN NB & SB DB's should be run in separate ovsdb-server processes
and should run with ovn-ctl start_northd / stop_northd
Signed-off-by: RYAN D. MOATS
Signed-off-by: Michael Arnaldi
---
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 169 +
From: RYAN D. MOATS
OVN NB & SB DB's should be run in separate ovsdb-server processes
and should run with ovn-ctl start_northd / stop_northd
Signed-off-by: RYAN D. MOATS
Signed-off-by: Michael Arnaldi
---
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 173 +
From: RYAN D. MOATS
Change from V6:
Split out NB and SB ovsdb process logfile as separate arguments to
avoid ugly quoting issues when trying to install via debian packages.
RYAN D. MOATS (1):
Separating OVN NB and SB database processes
ovn/northd/ovn-northd.c | 33 ++---
ovn/util
From: RYAN D. MOATS
Delta from V7: fixed typo in that V7 introduced in ovn-ctl
RYAN D. MOATS (1):
Separating OVN NB and SB database processes
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 173 +-
ovn/utilities/ovn-nbctl
From: RYAN D. MOATS
OVN NB & SB DB's should be run in separate ovsdb-server processes
and should run with ovn-ctl start_northd / stop_northd
Signed-off-by: RYAN D. MOATS
Signed-off-by: Michael Arnaldi
---
ovn/northd/ovn-northd.c | 33 ++---
ovn/utilities/ovn-ctl| 173 +
From: RYAN D. MOATS
V6->v7
Split out NB and SB ovsdb process logfiles as separate arguments
to avoid ugly quoting issues when trying to install via debian
packages.
v7->v8
Fixed typo introduced in ovn-ctl
v8->v9
Update debian/ovn-central.init script to ensure that stopping
ovn-central will stop
From: RYAN D. MOATS
OVN NB & SB DB's should be run in separate ovsdb-server processes
and should run with ovn-ctl start_northd / stop_northd
Signed-off-by: RYAN D. MOATS
Signed-off-by: Michael Arnaldi
---
debian/ovn-central.init |8 ++-
ovn/northd/ovn-northd.c | 33 ++---
Justin Pettit wrote on 03/04/2016 01:20:52 AM:
> From: Justin Pettit
> To: Russell Bryant
> Cc: Ryan Moats/Omaha/IBM@IBMUS, ovs dev ,
> arnaldimich...@gmail.com
> Date: 03/04/2016 01:21 AM
> Subject: Re: [ovs-dev] [PATCH v9 1/1] Separating OVN NB and SB
> database proce
From: RYAN D. MOATS
OVN NB & SB DB's should be run in separate ovsdb-server processes
and should run with ovn-ctl start_northd / stop_northd. This patch
includes changes to unit tests, tutorial and debian scripts to remain
self-consistent.
Signed-off-by: RYAN D. MOATS
Signed-off-by: Michael Ar
From: RYAN D. MOATS
Changes from V9:
Updated tutorial/ovs-sandbox to work with the split database processes
Addressed russellb's comments on V9:
- fixed vtep test case to split databases rather than change all the
ovn-*ctl calls
- fixed indentation on set_defaults()
- stop_ovsdb() now uses
finishes with a note describing the dummy
> environment it has set up. Add some additional text that indicates that
> OVN is also enabled when that is the case.
>
> Signed-off-by: Russell Bryant
Acked-by: Ryan Moats
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
All-
While working on finishing up the next version of the incremental flow
processing patch, I stubbed
my toe on a situation where the port column of rows in the multicast group
table of ovnsb were
being updated without the row's change seqno being updated.
A quick look at the code base shows t
Ben Pfaff wrote on 03/08/2016 10:07:19 PM:
> From: Ben Pfaff
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 03/08/2016 10:07 PM
> Subject: Re: [ovs-dev] What looks like a bug in OVSDB change seqno
tracking
>
> On Tue, Mar 08, 2016 at 08:35:10PM -0
From: RYAN D. MOATS
Change ovn-controller to use incremental processing in
encaps_run, binding_run, lflow_run and physical_run where
easily feasible (patch_run and the mutlicast group processing
in physical_run remain as complete rebuilds every cycle).
This patch includes all v7 comments.
Scale
From: RYAN D. MOATS
Currently changes are added to the front of the track list, so
they are looped through in LIFO order. Incremental processing
is more efficient with a FIFO presentation, so
(1) add new changes to the back of the track list, and
(2) move updated changes to the back of the track
From: RYAN D. MOATS
Modify test 1739 to output the OF flows from all three
hypervisors to help debug when something goes wrong.
Signed-off-by: RYAN D. MOATS
---
tests/ovn.at | 14 ++
1 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index
From: RYAN D. MOATS
Persist across runs so that a change to this simap can be used
as a trigger for resetting incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/lflow.c | 125
1 files changed, 115 insertions(+), 10 deletion
From: RYAN D. MOATS
This code changes lflow_run to do incremental process of the
logical flow table rather than processing the full table each run.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c|3 ++
ovn/controller/lflow.c | 53 +++
From: RYAN D. MOATS
Side effects include tunnel context being persisted and
no need to collect already defined OVS tunnels during each
execution.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/encaps.c | 123 +--
1 files changed, 66 insertions(+),
From: RYAN D. MOATS
Persist local_datapaths across runs so that a change can be used
as a trigger to reset incremental flow processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c| 41 --
ovn/controller/ovn-controller.c | 15 +++
From: RYAN D. MOATS
As lflow processing is incremental, reset it whenever a patch port
is added or removed.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/patch.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/ovn/controller/patch.c b/ovn/controller/patch.c
index
From: RYAN D. MOATS
This is a prerequisite for incremental processing.
Side effects:
1. Table rows are now tracked so that removed rows are correctly
handled.
2. Hash by table id+priority+action added to help detect superseded
flows.
3. Hash by insert seqno added to help find deleted flow
From: RYAN D. MOATS
Persist localvif_to_ofport and tunnels structures and change
physical_run to incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/lflow.c|3 +
ovn/controller/physical.c | 113 +++--
ovn/controller/physical.
From: RYAN D. MOATS
Persist all_lports structure and ensure that binding_run
resets to process the entire port binding table when chassis
are added/removed or when get_local_iface_ids finds new ports
on the local vswitch.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c | 50 +++
"dev" wrote on 03/09/2016 03:12:07 PM:
> From: Russell Bryant
> To: ovs dev
> Date: 03/09/2016 03:12 PM
> Subject: [ovs-dev] [RFC] OVN northbound address sets
> Sent by: "dev"
>
> I'd like to propose a new feature for the OVN northbound database. If we
> reach some consensus, I will impleme
"dev" wrote on 03/10/2016 01:11:09 AM:
> From: Ben Pfaff
> To: dev@openvswitch.org
> Date: 03/10/2016 01:31 AM
> Subject: [ovs-dev] RFC: OVN database options
> Sent by: "dev"
>
> Requirements
>
>
> OVN uses two databases, the "northbound" and "southbound" databases,
> in a somewha
From: RYAN D. MOATS
Currently changes are added to the front of the track list, so
they are looped through in LIFO order. Incremental processing
is more efficient with a FIFO presentation, so
(1) add new changes to the back of the track list, and
(2) move updated changes to the back of the track
From: RYAN D. MOATS
As lflow processing is incremental, reset it whenever a patch port
is added or removed.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/patch.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/ovn/controller/patch.c b/ovn/controller/patch.c
index
From: RYAN D. MOATS
Persist across runs so that a change to this simap can be used
as a trigger for resetting incremental processing.
---
ovn/controller/lflow.c | 125
1 files changed, 115 insertions(+), 10 deletions(-)
diff --git a/ovn/controll
From: RYAN D. MOATS
Persist localvif_to_ofport and tunnels structures and change
physical_run to incremental processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/lflow.c|3 +
ovn/controller/physical.c | 113 +++--
ovn/controller/physical.
From: RYAN D. MOATS
The delta from v8 is that patch 8 has been (a) rebased and (b) had a fix
added to address a bug found during scaling testing.
RYAN D. MOATS (10):
Add useful information to ovn E2E tests
Present tracked changes in increasing change number order
Make flow table persistent
From: RYAN D. MOATS
Persist all_lports structure and ensure that binding_run
resets to process the entire port binding table when chassis
are added/removed or when get_local_iface_ids finds new ports
on the local vswitch.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c| 55
From: RYAN D. MOATS
This is a prerequisite for incremental processing.
Side effects:
1. Table rows are now tracked so that removed rows are correctly
handled.
2. Hash by table id+priority+action added to help detect superseded
flows.
3. Hash by insert seqno added to help find deleted flow
From: RYAN D. MOATS
This code changes lflow_run to do incremental process of the
logical flow table rather than processing the full table each run.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c|3 ++
ovn/controller/lflow.c | 53 +++
From: RYAN D. MOATS
Persist local_datapaths across runs so that a change can be used
as a trigger to reset incremental flow processing.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/binding.c| 41 --
ovn/controller/ovn-controller.c | 15 +++
From: RYAN D. MOATS
Modify E2E test to output the OF flows from all three
hypervisors to help debug when something goes wrong.
Signed-off-by: RYAN D. MOATS
---
tests/ovn.at | 14 ++
1 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index
From: RYAN D. MOATS
Side effects include tunnel context being persisted and
no need to collect already defined OVS tunnels during each
execution.
Signed-off-by: RYAN D. MOATS
---
ovn/controller/encaps.c | 123 +--
1 files changed, 66 insertions(+),
n improve the OVSDB
> Server performance under certains loads.
>
> Signed-off-by: Esteban Rodriguez Betancourt
Acked-by: Ryan Moats
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
ovs-vsctl) to detect this
> condition and exit with an appropriate error message.
>
> Signed-off-by: Lance Richardson
> ---
> v2: changed ovsdb_idl_get_last_error() to better preserve the original
> semantics (per comments from B
> Currently, ``ovs-ctl start'' will attempt to start both the DB and
> vswitchd. This is quite convenient when the database already has all of
> the configuration values required, and when using a single services file
> for systemd integration. The same goes for the ``ovs-ctl stop'' command.
>
> H
"dev" wrote on 03/14/2016 12:57:10 PM:
> From: Ben Pfaff
> To: Guru Shetty
> Cc: ovs dev
> Date: 03/14/2016 12:57 PM
> Subject: Re: [ovs-dev] [RFC 0/8] OVN east-west loadbalancing.
> Sent by: "dev"
>
> On Mon, Feb 29, 2016 at 08:32:13AM -0800, Guru Shetty wrote:
> > On 28 February 2016 at 22:
From: RYAN D. MOATS
Allow TCP connection strings to also use DNS names in
addition to IPv4 and IPv6 addresses. Updated test case
"ovsdb-client get-schema-version - tcp socket" to
verify.
Signed-off-by: RYAN D. MOATS
---
lib/stream-tcp.c | 66 +
From: RYAN D. MOATS
Allows for auto detection and reconnect if the ovn-remote needs
to change. ovn-controller test case updated to include testing
this code
Signed-off-by: RYAN D. MOATS
---
ovn/controller/ovn-controller.c | 12
tests/ovn-controller.at | 16 +++
-Original Message
> @@ -256,8 +258,17 @@ physical_run(struct controller_ctx *ctx, enum
mf_field_id mff_ovn_geneve,
> } else {
> ofport = u16_to_ofp(simap_get(&localvif_to_ofport,
>binding->logical_port));
> -if
gt;
> VMware-BZ: #1598643
> Reported-by: Mike Qing
> Signed-off-by: Gurucharan Shetty
Acked-by: Ryan Moats
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Original Message
> Proposal: add an option to ovs-ofctl utility so as to obtain colorized
output
> in tty, for easier reading. Currently, only the dump-flows command
supports
> colors.
[Snipping for BW]
Gave this a test spin this morning and I like it!
Acked-by: R
Found a pair of places where the patch wasn't clean...
original message
> diff --git a/datapath/linux/compat/include/net/ip.h
b/datapath/linux/compat/include/net/ip.h
> index cd87bcc..b749301 100644
> --- a/datapath/linux/compat/include/net/ip.h
> +++ b/datapath/linux/compat/include/net
"dev" wrote on 03/15/2016 11:21:16 AM:
> From: Ryan Moats/Omaha/IBM@IBMUS
> To: "ovs dev"
> Date: 03/15/2016 11:22 AM
> Subject: Re: [ovs-dev] [ovs-dev, RESEND] openvswitch: add support
> for kernel 4.4
> Sent by: "dev"
>
>
Original Message -
> To allow this variable be updated from multiple threads.
>
> Signed-off-by: Andy Zhou
Acked-by: Ryan Moats
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
1 - 100 of 765 matches
Mail list logo