[ovs-dev] [PATCH] vtep: add source node replication support.

2016-04-14 Thread Darrell Ball
. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 16 + vtep/README.ovs-vtep.md | 18 +-- vtep/ovs-vtep | 55 + vtep/vtep-ctl.8.in | 20 + vtep/vtep-ctl.c | 60

[ovs-dev] [PATCH] vtep: add source node replication support.

2016-04-14 Thread Darrell Ball
to read the vtep database physical and logical switch replication modes to support both types of replication. The vtep emulator help file is also updated. Darrell Ball (1): vtep: add source node replication support. tests/vtep-ctl.at | 16 + vtep/README.ovs-vtep.md | 18

Re: [ovs-dev] [PATCH] vtep: add source node replication support.

2016-04-15 Thread Darrell Ball
e", but thought it might confuse some readers and then settled on incomplete wording instead :-) > > At this point I’ve only reviewed vtep.ovsschema and vtep.xml. Hope this > makes sense & helps. > It does; thank you > > Bruce > > > > On Apr 14, 2

Re: [ovs-dev] [PATCH] vtep: add source node replication support.

2016-04-15 Thread Darrell Ball
ent > to the minor version, so the last line should be “1.6.0" > > Bruce > > On Apr 15, 2016, at 3:47 PM, Darrell Ball wrote: > > Thank you Bruce for your feedback and taking the time > > > ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [patch_v2] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 17 +++ vtep/README.ovs-vtep.md | 19 +++-- vtep/ovs-vtep | 31

[ovs-dev] [patch_v2] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
e is the default replication mode, v2 makes replication mode an option and only source node mode can be optionally configured. If a physical switch cannot support source node replication, then an error should be set in the physical switch table that can be read by a controller. Darrell Ball (1):

[ovs-dev] [patch_v3] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 17 +++ vtep/README.ovs-vtep.md | 19 +++-- vtep/ovs-vtep | 36

[ovs-dev] [patch_v3] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
couple bugs in ovs-vtep I noticed after v2. Darrell Ball (1): vtep: add source node replication support. tests/vtep-ctl.at | 17 +++ vtep/README.ovs-vtep.md | 19 +++-- vtep/ovs-vtep | 36 +++ vtep/vtep-ctl.8.in | 9 ++

Re: [ovs-dev] [patch_v2] vtep: add source node replication support.

2016-04-25 Thread Darrell Ball
This patch is superseded I sent an updated patch with a couple bugs fixed in ovs-vtep Darrell On Mon, Apr 25, 2016 at 1:43 PM, Darrell Ball wrote: > This patch series updates the vtep schema, vtep-ctl commands and vtep > simulator to support source node replication in addition to servic

Re: [ovs-dev] [PATCH] Patch v3: ovn-controller-vtep: Support BUM traffic for the VTEP Schema

2016-04-25 Thread Darrell Ball
Thanks for the review; The suggestions are fine; I added a couple comments inline. Darrell On Mon, Apr 25, 2016 at 3:44 PM, Justin Pettit wrote: > > > On Apr 5, 2016, at 1:13 PM, Darrell Ball wrote: > > > > This patch implements BUM support in the VTEP schema.

Re: [ovs-dev] [patch_v3] vtep: add source node replication support.

2016-04-26 Thread Darrell Ball
Thanks Bruce I have one point that I would like to confirm. On Tue, Apr 26, 2016 at 11:41 AM, Bruce Davie wrote: > Darrell, > I don’t think you can handle the error of unsupported replication mode > the way it is described here: > > + +key="unsupported_source_node_replication"> >

[ovs-dev] [patch_v4] vtep: add source node replication support.

2016-04-26 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 17 ++ vtep/README.ovs-vtep.md | 19 ++-- vtep/ovs-vtep | 36

[ovs-dev] [patch_v4] vtep: add source node replication support.

2016-04-26 Thread Darrell Ball
plication mode support. Reworded alternate replication mode description in vtep.xml per Bruce. Shorten name to Alternate Replication Mode from per logical switch Alternate Replication Mode. Darrell Ball (1): vtep: add source node replication support. tests/vtep-ctl.at | 17 +

[ovs-dev] [patch_v5] vtep: add source node replication support.

2016-04-27 Thread Darrell Ball
atus based on self review. Fixed a line length violation in ovs-vtep reported by checkpatch. Darrell Ball (1): vtep: add source node replication support. tests/vtep-ctl.at | 17 ++ vtep/README.ovs-vtep.md | 19 ++-- vtep/ovs-vtep | 37 +

[ovs-dev] [patch_v5] vtep: add source node replication support.

2016-04-27 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 17 ++ vtep/README.ovs-vtep.md | 19 ++-- vtep/ovs-vtep | 37

[ovs-dev] [patch_v6] vtep: add source node replication support.

2016-04-27 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 17 ++ vtep/README.ovs-vtep.md | 19 ++-- vtep/ovs-vtep | 37

[ovs-dev] [patch_v6] vtep: add source node replication support.

2016-04-27 Thread Darrell Ball
lt_status based on self review. Fixed a line length violation in ovs-vtep reported by checkpatch. v5->v6: Changes from more review from Bruce (offline). Removed duplicate "to" in comment in ovs-vtep. Added a comma to vtep.xml for Alternate Replication Mode description. Darrell Ball (1):

Re: [ovs-dev] [PATCH V5 1/1] ovn-northd: Add support for static_routes.

2016-04-28 Thread Darrell Ball
On Sun, Apr 24, 2016 at 2:49 AM, steve.ruan wrote: > static routes are useful when connecting multiple > routers with each other. > Logical patch ports are used to connect logical routers together. Static routes are used to select between different logical router ports when exiting a logical ro

[ovs-dev] [patch_v7] vtep: add source node replication support.

2016-04-28 Thread Darrell Ball
below. - if not replication_mode: +if replication_mode == "[]": Darrell Ball (1): vtep: add source node replication support. tests/vtep-ctl.at | 17 ++ vtep/README.ovs-vtep.md | 19 ++-- vtep/ovs-vtep | 37 ++

[ovs-dev] [patch_v7] vtep: add source node replication support.

2016-04-28 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 17 ++ vtep/README.ovs-vtep.md | 19 ++-- vtep/ovs-vtep | 37

Re: [ovs-dev] [patch_v7] vtep: add source node replication support.

2016-04-29 Thread Darrell Ball
On Fri, Apr 29, 2016 at 2:16 PM, Justin Pettit wrote: > > > On Apr 28, 2016, at 8:46 PM, Darrell Ball wrote: > > > > This patch series updates the vtep schema, vtep-ctl commands and vtep > > simulator to support source node replication in addition to service node

Re: [ovs-dev] [PATCH] system-traffic: Check namespace exists befoe delete.

2016-04-30 Thread Darrell Ball
On Fri, Apr 29, 2016 at 11:11 PM, William Tu wrote: > This avoids error message "Cannot remove namespace file > "/var/run/netns/at_ns0": No such file or directory", > when at_ns0 does not exist. > Signed-off-by: William Tu > --- > tests/system-common-macros.at | 6 -- > 1 file changed, 4

Re: [ovs-dev] [PATCH] system-traffic: Check namespace exists befoe delete.

2016-05-02 Thread Darrell Ball
On Mon, May 2, 2016 at 8:39 AM, William Tu wrote: > Hi Darrel, > > # Delete namespaces from the running OS >>> m4_define([DEL_NAMESPACES], >>> [m4_foreach([ns], [$@], >>> - [ip netns del ns >>> -]) >>> + [if ip netns list | grep ns > /dev/null; then >>> +

Re: [ovs-dev] [PATCH] vtep/ovs-vtep: support multiple ovs-vtep processes which using the isolated logical-switch

2016-05-02 Thread Darrell Ball
On Sun, May 1, 2016 at 4:15 PM, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > Hi all, > The computer can run multiple ovs-vtep processes, but ovs-vtep > processes share the “vtep_lsX” bridge when binding the physical-switch and > logical-switch. The bridge crea

Re: [ovs-dev] [PATCH] system-traffic: Check namespace exists befoe delete.

2016-05-02 Thread Darrell Ball
On Mon, May 2, 2016 at 11:12 AM, Joe Stringer wrote: > On 2 May 2016 at 09:04, Darrell Ball wrote: > > On Mon, May 2, 2016 at 8:39 AM, William Tu wrote: > > > >> Hi Darrel, > >> > >> # Delete namespaces from the running OS > >>>>

Re: [ovs-dev] [PATCH] system-traffic: Check namespace exists befoe delete.

2016-05-02 Thread Darrell Ball
less code -> less bugs On Mon, May 2, 2016 at 5:45 PM, Joe Stringer wrote: > On 2 May 2016 at 16:57, Darrell Ball wrote: > > > > > > On Mon, May 2, 2016 at 11:12 AM, Joe Stringer wrote: > >> > >> On 2 May 2016 at 09:04, Darrell Ball wrote: > &g

Re: [ovs-dev] [PATCH V5 1/1] ovn-northd: Add support forstatic_routes.

2016-05-03 Thread Darrell Ball
Hi Steve Your e-mail content was filtered by the mailing list and hence my response was also filtered. I added the context below. Thanks Darrell On Tue, May 3, 2016 at 3:36 AM, Shi Xin Ruan wrote: > Hi Darrell, > > > Thanks for the comments. > Sorry for the latest reply because I had a long

[ovs-dev] [patch_v8] vtep: add source node replication support.

2016-05-03 Thread Darrell Ball
set-replication-mode and also for other similar commands; i.e. removed "-ls" portion. Added a get-replication-mode command and added it to vtep-ctl tests. Moved all replication mode information in the ovs-vtep readme to a single step. Darrell Ball (1): vtep: add source node replicatio

[ovs-dev] [patch_v8] vtep: add source node replication support.

2016-05-03 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 32 +++ vtep/README.ovs-vtep.md | 30 -- vtep/ovs-vtep

Re: [ovs-dev] [patch_v8] vtep: add source node replication support.

2016-05-03 Thread Darrell Ball
Thanks Anupam Darrell On Tue, May 3, 2016 at 4:28 PM, Anupam Chanda wrote: > Hi Darrell, > > > Looks good, thanks for working on this. Some comments below tagged with > [AC]. > > > Thanks, > > Anupam > > From: Darrell Ball > Date: Tue, May 3, 2016 at

Re: [ovs-dev] [patch_v8] vtep: add source node replication support.

2016-05-03 Thread Darrell Ball
Thanks Anupam I'll send the updated patch. Darrell On Tue, May 3, 2016 at 7:09 PM, Anupam Chanda wrote: > Hi Darrell, > > Inlined below with [AC]. > > Thanks, > Anupam > > > Original message > From: Darrell Ball > Date: 5/3/2016 5:35 PM

Re: [ovs-dev] [PATCH V5 1/1] ovn-northd: Add support forstatic_routes.

2016-05-03 Thread Darrell Ball
ssage -- From: Darrell Ball To: Shi Xin Ruan Cc: ovs dev , "steve.ruan" Date: Tue, 3 May 2016 19:17:31 -0700 Subject: Re: [ovs-dev] [PATCH V5 1/1] ovn-northd: Add support forstatic_routes. Hi Steve On Tue, May 3, 2016 at 7:00 PM, Shi Xin Ruan wrote: > Hi Darrel, > >

Re: [ovs-dev] [patch_v8] vtep: add source node replication support.

2016-05-04 Thread Darrell Ball
On Wed, May 4, 2016 at 8:26 AM, Ryan Moats wrote: > "dev" wrote on 05/03/2016 05:32:37 PM: > > > From: Darrell Ball > > To: dlu...@gmail.com, bda...@vmware.com, acha...@vmware.com, > > d...@openvswitch.com > > Date: 05/03/2016 05:33 PM > > Subje

[ovs-dev] [patch_v9] vtep: add source node replication support.

2016-05-04 Thread Darrell Ball
ing in a few places. Omit invalidate cache in get-replication-mode, as its unnecessary. Add replication modes description to ovs-vtep help file and vtep-ctl man page, in addition to previous vtep.xml file. I also added another test for vtep-ctl to make the behavior more explicit. Darrell

[ovs-dev] [patch_v9] vtep: add source node replication support.

2016-05-04 Thread Darrell Ball
is optionally configurable and resetting the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 43 +++ vtep/README.ovs-vtep.md | 49

Re: [ovs-dev] [PATCH] system-traffic: Remove DEL_NAMESPACE at ADD_NAMESPACE.

2016-05-04 Thread Darrell Ball
On Wed, May 4, 2016 at 3:04 PM, William Tu wrote: > Since in ADD_NAMESPACE we've queued up namespace deletion using on_exit, > whether the test passes or fails, the namespace will be cleaned up. So > remove the DEL_NAMESPACE before adding the namespace. > > Signed-off-by: William Tu > --- > te

Re: [ovs-dev] [patch_v9] vtep: add source node replication support.

2016-05-06 Thread Darrell Ball
On Fri, May 6, 2016 at 5:57 PM, Justin Pettit wrote: > > > On May 4, 2016, at 12:03 PM, Darrell Ball wrote: > > > > This patch series updates the vtep schema, vtep-ctl commands and vtep > > It's not really a series. Even if it were, when looking at the commit

Re: [ovs-dev] [patch_v9] vtep: add source node replication support.

2016-05-07 Thread Darrell Ball
f the VTEP schema. On Fri, May 6, 2016 at 9:07 PM, Justin Pettit wrote: > > > On May 6, 2016, at 7:44 PM, Darrell Ball wrote: > > > > > +A return value of NULL indicates the replication mode column is not > set > > > +and therefore a default of "se

[ovs-dev] [patch_v10] vtep: add source node replication support.

2016-05-07 Thread Darrell Ball
et-replication-mode as the word is contentious. Remove some unnecessary words. Reorder 2 steps in ovs-vtep README. Darrell Ball (1): vtep: add source node replication support. tests/vtep-ctl.at | 32 +++ vtep/README.ovs-vtep.md | 28 +++-- vtep/ovs

[ovs-dev] [patch_v10] vtep: add source node replication support.

2016-05-07 Thread Darrell Ball
optionally configurable and clearing the replication mode implicitly sets the replication mode back to a default of service node. Signed-off-by: Darrell Ball --- tests/vtep-ctl.at | 32 +++ vtep/README.ovs-vtep.md | 28 +++-- vtep/ovs-vtep | 36

[ovs-dev] [patch_v1] ovn: add local router support (RFC)

2016-05-09 Thread Darrell Ball
router to be translated into openflow rules and programmed on this chassis. If a logical router is labelled as local but a given chassis is not configured for that logical router to be local, then the corresponding openflow rules are not programmed on this chassis. Signed-off-by: Darrell Ball

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-09 Thread Darrell Ball
I have some initial clarifications first before review On Fri, May 6, 2016 at 9:21 AM, Gurucharan Shetty wrote: > Currently we can connect routers via "peer"ing. This limits > the number of routers that can be connected with each other > directly to 2. > This sounds like you are saying that ca

[ovs-dev] [patch_v2] ovn: add local router support (RFC)

2016-05-09 Thread Darrell Ball
in test avoid creating patch port where unnecessary Signed-off-by: Darrell Ball --- ovn/controller/chassis.c| 32 +- ovn/controller/chassis.h| 4 +- ovn/controller/lflow.c | 16 ++- ovn/controller/lflow.h | 4 +- ovn/controller/

[ovs-dev] [patch_v3] ovn: add local router support (RFC)

2016-05-09 Thread Darrell Ball
router to be translated into openflow rules and programmed on this chassis. If a logical router is labelled as local but a given chassis is not configured for that logical router to be local, then the corresponding openflow rules are not programmed on this chassis. Signed-off-by: Darrell Ball v1

Re: [ovs-dev] [patch_v2] ovn: add local router support (RFC)

2016-05-09 Thread Darrell Ball
superceded by v3 On Mon, May 9, 2016 at 8:53 PM, Darrell Ball wrote: > This patch adds local router support. A logical router can > be declared local at northbound level via external_ids. > This is communicated to the southbound database via northd > which populates the extern

Re: [ovs-dev] [patch_v1] ovn: add local router support (RFC)

2016-05-09 Thread Darrell Ball
hv1 is R1 routes. If I change the test with new permissive catchall routes everywhere, I could do it, of course, but that is something else and contrary to what you would want to do. > > Mickey > > > -----"dev" wrote: - > To: dlu...@gmail.com, d...@openvswitch.com

Re: [ovs-dev] [patch_v9] vtep: add source node replication support.

2016-05-09 Thread Darrell Ball
On Sun, May 8, 2016 at 8:59 AM, Ben Pfaff wrote: > On Fri, May 06, 2016 at 07:44:57PM -0700, Darrell Ball wrote: > > On Fri, May 6, 2016 at 5:57 PM, Justin Pettit wrote: > > > > > > > > > On May 4, 2016, at 12:03 PM, Darrell Ball wrote: > > > > &

Re: [ovs-dev] [patch_v1] ovn: add local router support (RFC)

2016-05-10 Thread Darrell Ball
s too far, I would like to point out that I have > done something similar to what Mickey is suggesting here for the l3 gateway > work. I sent a patch here: > http://openvswitch.org/pipermail/dev/2016-May/070790.html > > I will send the DNAT and SNAT implementations on top of the above p

Re: [ovs-dev] [patch_v10] vtep: add source node replication support.

2016-05-10 Thread Darrell Ball
On Tue, May 10, 2016 at 1:39 PM, Justin Pettit wrote: > > > On May 7, 2016, at 9:21 AM, Darrell Ball wrote: > > > > This patch updates the vtep schema, vtep-ctl commands and vtep simulator > > to support source node replication in addition to service node > repl

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-11 Thread Darrell Ball
On Tue, May 10, 2016 at 7:38 AM, Guru Shetty wrote: > > > On 9 May 2016 at 14:51, Darrell Ball wrote: > >> I have some initial clarifications first before review >> >> On Fri, May 6, 2016 at 9:21 AM, Gurucharan Shetty wrote: >> >>> Currently we c

Re: [ovs-dev] [patch_v10] vtep: add source node replication support.

2016-05-11 Thread Darrell Ball
I am fine with the patch Justin; it can be merged. Thanks Darrell On Wed, May 11, 2016 at 4:20 PM, Justin Pettit wrote: > > > On May 10, 2016, at 4:07 PM, Darrell Ball wrote: > > > > > > > > On Tue, May 10, 2016 at 1:39 PM, Justin Pettit wrote: > > &g

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-11 Thread Darrell Ball
On Wed, May 11, 2016 at 4:42 PM, Guru Shetty wrote: > >> >> Some reasons why having a “transit LS” is “undesirable” is: >> >> 1)1) It creates additional requirements at the CMS layer for setting >> up networks; i.e. additional programming is required at the OVN northbound >> interface for th

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-11 Thread Darrell Ball
On Wed, May 11, 2016 at 8:51 PM, Guru Shetty wrote: > > > > > > On May 11, 2016, at 8:45 PM, Darrell Ball wrote: > > > >> On Wed, May 11, 2016 at 4:42 PM, Guru Shetty wrote: > >> > >> > >>> > >>> Some reasons why

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-12 Thread Darrell Ball
On Thu, May 12, 2016 at 6:03 AM, Guru Shetty wrote: > > > > On May 11, 2016, at 10:45 PM, Darrell Ball wrote: > > > > On Wed, May 11, 2016 at 8:51 PM, Guru Shetty wrote: > >> >> >> >> >> > On May 11, 2016, at 8:45 PM, Darrell Ball wr

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-12 Thread Darrell Ball
On Thu, May 12, 2016 at 10:54 AM, Guru Shetty wrote: > >> I think you misunderstood - having one or more gateway per tenant does >> not make Transit LS better in flow scale. >> The size of a Transit LS subnet and management across Transit LSs is one >> the 5 issues I mentioned and it remains the

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-12 Thread Darrell Ball
On Thu, May 12, 2016 at 4:55 PM, Guru Shetty wrote: > > > On 12 May 2016 at 16:34, Darrell Ball wrote: > >> On Thu, May 12, 2016 at 10:54 AM, Guru Shetty wrote: >> >> > >> >> I think you misunderstood - having one or more gateway per tenant does &

Re: [ovs-dev] [PATCH] pinctrl: Fix "sparse" warning.

2016-05-17 Thread Darrell Ball
On Tue, May 17, 2016 at 7:44 AM, Ben Pfaff wrote: > The ofport member should be an ofp_port_t, since it represents an OpenFlow > port number. > > Fixes: 9baaabfff3c7 ("ovn: Fix localnet ports deletion and recreation > sometimes after restart.") > Is this rather related to "ovn: send garp on loca

Re: [ovs-dev] [PATCH monitor_cond V6 11/11] RFC OVN: Quick implementation of conditional monitoring

2016-05-17 Thread Darrell Ball
On Tue, May 17, 2016 at 7:27 AM, Liran Schour wrote: > Conditional monitor of: Port_Binding, Logical_Flow, Multicast_Group > MAC_Binding tables. As a result ovn-controller will be notified only about > records belongs to a datapath that is being served by this hypervisor. > > Hack: Ideally, logic

Re: [ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

2016-05-21 Thread Darrell Ball
On Thu, May 12, 2016 at 5:59 PM, Guru Shetty wrote: > >> >> >> >> With respect to the other questions, I think its best approach would >> be to >> >> ask direct questions so those >> >> direct questions get answered. >> >> >> >> 1) With 1000 HVs, 1000 HVs/tenant, 1 distributed router per tenant,

Re: [ovs-dev] [PATCH v3 2/5] ovn: Introduce l3 gateway router.

2016-05-21 Thread Darrell Ball
I made some modifications to code in Patches 1 and 2 to remove the Transit LS requirements. In summary: I removed all changes to lflow.c thereby reinstating the previous optimization. I made some modifications to ovn-northd.c changes to remove the Transit LS special aspects and additional arp fl

Re: [ovs-dev] [PATCH v3 2/5] ovn: Introduce l3 gateway router.

2016-05-24 Thread Darrell Ball
On Tue, May 24, 2016 at 7:41 AM, Guru Shetty wrote: > > > On 21 May 2016 at 11:48, Darrell Ball wrote: > >> I made some modifications to code in Patches 1 and 2 to remove the Transit >> LS >> requirements. >> > > These are the reasons for the

Re: [ovs-dev] [PATCH v3 2/5] ovn: Introduce l3 gateway router.

2016-05-24 Thread Darrell Ball
Correcting typo below Also, supporting transit LS should "NOT" prevent other optimizations. On Tue, May 24, 2016 at 3:17 PM, Darrell Ball wrote: > > > On Tue, May 24, 2016 at 7:41 AM, Guru Shetty wrote: > >> >> >> On 21 May 2016 at 11:48,

Re: [ovs-dev] Question on L3 Logical Router in vtep-emulator

2016-05-26 Thread Darrell Ball
On Thu, May 26, 2016 at 9:14 AM, Kyle Mestery wrote: > On Thu, May 26, 2016 at 10:46 AM, Guru Shetty wrote: > > > >> > >> > >> OK, I'm going to play with this a bit. I was mostly interested in > >> feedback from others. We're interested in this approach and so if we > >> take the patch up does i

Re: [ovs-dev] [PATCH v2] ovn-northd: logical router icmp response should not care about inport

2016-05-26 Thread Darrell Ball
On Wed, May 25, 2016 at 9:40 PM, Flavio Fernandes wrote: > When responding to icmp echo requests (aka ping) packets, the logical > router should not restrict responses based on the inport. > > Example diagram: > > vm: IP1.1 (subnet1) > logical_router: IP1.2 (subnet1) and IP2.2 (subnet2) > >vm

Re: [ovs-dev] [PATCH] ovs-vtep: Make compatible with python2.7 and 3.

2016-05-27 Thread Darrell Ball
On Mon, May 23, 2016 at 6:11 PM, Joe Stringer wrote: > Translate commandline calls to UTF-8, appease flake8 and use six's > integer types. This allows the testsuite to pass when using python3 as > your default system python version. > > Signed-off-by: Joe Stringer >

[ovs-dev] [patch_v4] ovn: Add local router support (RFC)

2016-05-31 Thread Darrell Ball
southbound logical flows that are needed on a given HV. Two hash tables are used to keep track of which logical router data paths are needed locally and which are already known to be not needed. Signed-off-by: Darrell Ball --- ovn/controller/lflow.c | 113 +++ ovn/northd/ovn

[ovs-dev] [patch_v1] ovn: Fix receive from vxlan in ovn-controller.

2016-06-01 Thread Darrell Ball
register usage. As part of this change ovn-controller-vtep is hard-coded to set the replication mode of each logical switch to source node as OVN will only support source node replication. Signed-off-by: Darrell Ball --- ovn/OVN-DESIGN.md | 170

Re: [ovs-dev] Question about ovs-vtep implementation

2016-07-12 Thread Darrell Ball
On Fri, Jul 8, 2016 at 12:20 PM, Lance Richardson wrote: > The "ovn-controller-vtep - vtep-macs 1" test case fails occasionally, > with ovs-vswitchd logs similar to these: > > bridge|INFO|bridge br-vtep_vtep_ls1: added interface vx1 on port 2 > tunnel|WARN|bfd1.2.3.5: attempting to add tu

Re: [ovs-dev] [PATCH] ovn-controller-vtep: occasional failure in "binding 1" test case

2016-07-12 Thread Darrell Ball
On Fri, Jul 8, 2016 at 7:16 AM, Lance Richardson wrote: > The ovn-controller-vtep "binding 1" test case fails occasionally > due to a race with the ovs-vtep daemon. If ovs-vtep happens to > execute handle_physical() after "ovs-vsctl del-port p0", but before > the test script has executed "vtep-c

Re: [ovs-dev] [PATCH] ovn-controller-vtep: occasional failure in "binding 1" test case

2016-07-14 Thread Darrell Ball
On Wed, Jul 13, 2016 at 2:15 PM, Lance Richardson wrote: > > > diff --git a/tests/ovn-controller-vtep.at b/tests/ > ovn-controller-vtep.at > > > index c296f0e..c978137 100644 > > > --- a/tests/ovn-controller-vtep.at > > > +++ b/tests/ovn-controller-vtep.at > > > @@ -225,8 +225,9 @@ AT_CHECK([sed

[ovs-dev] [patch_v1] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Darrell Ball
Presently, ovs-vtep expects the datapath tunnel key to be available in the VTEP DB at startup. This may not be the case which is also observed as interrmittent unit test failures. This patch allows for the tunnel key to later appear in the VTEP database. Signed-off-by: Darrell Ball --- vtep/ovs

[ovs-dev] [patch_v2] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Darrell Ball
Presently, ovs-vtep expects the datapath tunnel key to be available in the VTEP DB at startup. This may not be the case which is also observed as interrmittent unit test failures. This patch allows for the tunnel key to later appear in the VTEP database. Signed-off-by: Darrell Ball --- v1->

Re: [ovs-dev] [patch_v1] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Darrell Ball
On Thu, Jul 14, 2016 at 1:51 PM, Russell Bryant wrote: > > On Thu, Jul 14, 2016 at 4:09 PM, Darrell Ball wrote: > >> Presently, ovs-vtep expects the datapath tunnel key to be available >> in the VTEP DB at startup. This may not be the case which is also >> observed

Re: [ovs-dev] [PATCH v2] ovn-controller: Clean up bindings handling.

2016-07-14 Thread Darrell Ball
On Thu, Jul 14, 2016 at 12:47 PM, Russell Bryant wrote: > Remove the global set of logical port IDs called 'all_lports'. This is > no longer used for anything after conntrack ID assignment was moved out > of binding.c. > > Remove the global smap of logical port IDs to ovsrec_interface records. >

Re: [ovs-dev] [PATCH v2] ovn-controller-vtep: occasional failure in "binding 1" test case

2016-07-15 Thread Darrell Ball
for ovs-vtep to remove > entries for the deleted physical ports from the vtep db. > > Signed-off-by: Lance Richardson > Suggested-by: Darrell Ball > --- > tests/ovn-controller-vtep.at | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/o

Re: [ovs-dev] Question about ovs-vtep implementation

2016-07-15 Thread Darrell Ball
On Fri, Jul 15, 2016 at 11:00 AM, Lance Richardson wrote: > > From: "Darrell Ball" > > To: "Lance Richardson" > > Cc: "ovs dev" > > Sent: Tuesday, July 12, 2016 5:34:04 PM > > Subject: Re: [ovs-dev] Question about ovs-vtep i

Re: [ovs-dev] [PATCH monitor_cond V10] RFC OVN: Implementation of conditional monitoring usage

2016-07-22 Thread Darrell Ball
On Thu, Jul 21, 2016 at 4:18 AM, Liran Schour wrote: > Mickey Spiegel/San Jose/IBM wrote on 20/07/2016 08:53:42 AM: > > > From: Mickey Spiegel/San Jose/IBM > > To: Liran Schour/Haifa/IBM@IBMIL > > Cc: Ben Pfaff , dev@openvswitch.org > > Date: 20/07/2016 08:53 AM > > Subject: Re: [ovs-dev] [PATCH

[ovs-dev] [patch_v6 0/2] ovn: Fix receive from vxlan in ovn-controller.

2016-07-23 Thread Darrell Ball
. Split code change for bug fix and documentation update into two patches. v1->v2: Rebased after recent conflicting commit. Converted some xml comments ported from the ovn-architecture document. Removed redundant register initialization and unnecessary bit declaration. Darrell Ball (2): ovn:

[ovs-dev] [patch_v6 1/2] ovn: Fix receive from vxlan in ovn-controller.

2016-07-23 Thread Darrell Ball
vxlan context. As part of this change ovn-controller-vtep is hard-coded to set the replication mode of each logical switch to source node as OVN will only support source node replication. Signed-off-by: Darrell Ball --- ovn/controller-vtep/vtep.c | 4 ovn/controller/physical.c | 25

[ovs-dev] [patch_v6 2/2] ovn: Reformat some ovn design documentation.

2016-07-23 Thread Darrell Ball
following markdown parsers: https://jbt.github.io/markdown-editor/ http://dillinger.io/ The new flags register usage is also documented in the new OVN-DESIGN.md document. Signed-off-by: Darrell Ball --- ovn/OVN-DESIGN.md | 199 + ovn/automake.mk

[ovs-dev] [patch_v1] ovn: Add datapaths of interest filtering (RFC).

2016-07-29 Thread Darrell Ball
area. Signed-off-by: Darrell Ball --- ovn/controller/binding.c| 21 +--- ovn/controller/ovn-controller.c | 116 ++-- ovn/controller/ovn-controller.h | 8 +++ ovn/controller/patch.c | 19 +-- ovn/northd/ovn-northd.c | 74

[ovs-dev] [patch_v7] ovn: Fix receive from vxlan in ovn-controller.

2016-07-29 Thread Darrell Ball
to set the replication mode of each logical switch to source node as OVN will only support source node replication. Signed-off-by: Darrell Ball --- ovn/controller-vtep/vtep.c | 4 ovn/controller/physical.c | 25 - ovn/lib/logical-fields.h | 15 +++ ovn/ovn-arch

Re: [ovs-dev] [patch_v6 1/2] ovn: Fix receive from vxlan in ovn-controller.

2016-07-29 Thread Darrell Ball
On Wed, Jul 27, 2016 at 1:35 PM, Ben Pfaff wrote: > On Sat, Jul 23, 2016 at 01:18:01PM -0700, Darrell Ball wrote: > > This patch enables source node replication in OVN for receive from Vxlan > > tunnels. OVN only supports source node replication mode. > > > > OV

Re: [ovs-dev] [patch_v1] ovn: Add datapaths of interest filtering (RFC).

2016-07-31 Thread Darrell Ball
On Sat, Jul 30, 2016 at 10:58 AM, Liran Schour wrote: > "dev" wrote on 29/07/2016 11:46:09 AM: > > > From: Darrell Ball > > To: dlu...@gmail.com, d...@openvswitch.com, b...@ovn.org > > Date: 29/07/2016 11:46 AM > > Subject: [ovs-dev] [patch_v1] ovn: Ad

Re: [ovs-dev] [PATCH] ovn: Add second ACL stage

2016-08-02 Thread Darrell Ball
On Tue, Aug 2, 2016 at 4:52 AM, Russell Bryant wrote: > On Sat, Jul 30, 2016 at 4:19 PM, Mickey Spiegel > wrote: > > > On Fri, Jul 29, 2016 at 10:28 AM, Mickey Spiegel > > wrote: > > > > > > -"dev" wrote: - > > >> To: Mickey Spiegel > > >> From: Russell Bryant > > >> Sent by: "dev" >

Re: [ovs-dev] [PATCH] ovn: Add second ACL stage

2016-08-02 Thread Darrell Ball
On Tue, Aug 2, 2016 at 10:23 AM, Mickey Spiegel wrote: > On Tue, Aug 2, 2016 at 9:26 AM, Darrell Ball wrote: > >> >> >> On Tue, Aug 2, 2016 at 4:52 AM, Russell Bryant wrote: >> >>> On Sat, Jul 30, 2016 at 4:19 PM, Mickey Spiegel >>> wrote: &g

Re: [ovs-dev] [PATCH] ovn: Add second ACL stage

2016-08-02 Thread Darrell Ball
On Tue, Aug 2, 2016 at 12:05 PM, Russell Bryant wrote: > > > On Tue, Aug 2, 2016 at 3:02 PM, Darrell Ball wrote: > >> >> >> On Tue, Aug 2, 2016 at 10:23 AM, Mickey Spiegel >> wrote: >> >>> On Tue, Aug 2, 2016 at 9:26 AM, Darrell Ball wrote

Re: [ovs-dev] [PATCH] ovn: Add second ACL stage

2016-08-02 Thread Darrell Ball
On Tue, Aug 2, 2016 at 2:38 PM, Mickey Spiegel wrote: > On Tue, Aug 2, 2016 at 1:39 PM, Darrell Ball wrote: > >> >> >> On Tue, Aug 2, 2016 at 12:05 PM, Russell Bryant wrote: >> >>> >>> >>> On Tue, Aug 2, 2016 at 3:02 PM, Darrell Ball

[ovs-dev] [patch_v2] ovn: Add datapaths of interest filtering.

2016-08-02 Thread Darrell Ball
- related_datapaths to facilitate this so all datapaths associations are known quickly in ovn-controller. This allows monitoring to adapt quickly with a single new monitor setting for all datapaths of interest locally. Signed-off-by: Darrell Ball --- v1->v2: Added logical port removal monitor

[ovs-dev] [patch_v3] ovn: Add datapaths of interest filtering.

2016-08-02 Thread Darrell Ball
- related_datapaths to facilitate this so all datapaths associations are known quickly in ovn-controller. This allows monitoring to adapt quickly with a single new monitor setting for all datapaths of interest locally. Signed-off-by: Darrell Ball --- v2->v3: Line length violation fixups :/ v1-

Re: [ovs-dev] [patch_v7] ovn: Fix receive from vxlan in ovn-controller.

2016-08-08 Thread Darrell Ball
On Sat, Aug 6, 2016 at 12:38 PM, Ryan Moats wrote: > "dev" wrote on 07/29/2016 06:08:38 PM: > > > From: Darrell Ball > > To: dlu...@gmail.com, d...@openvswitch.com, b...@ovn.org > > Date: 07/29/2016 06:09 PM > > Subject: [ovs-dev] [patch_v7] ovn: Fi

Re: [ovs-dev] [patch_v7] ovn: Fix receive from vxlan in ovn-controller.

2016-08-08 Thread Darrell Ball
closely with some folks. On Mon, Aug 8, 2016 at 12:57 PM, Darrell Ball wrote: > > > On Sat, Aug 6, 2016 at 12:38 PM, Ryan Moats wrote: > >> "dev" wrote on 07/29/2016 06:08:38 PM: >> >> > From: Darrell Ball >> > To: dlu...@gmail.com, d...@openvswi

[ovs-dev] [patch_v8] ovn: Fix receive from vxlan in ovn-controller.

2016-08-08 Thread Darrell Ball
state for receipt from a vxlan tunnel, a MFF logical flags register flag is allocated. As part of this change ovn-controller-vtep is hard-coded to set the replication mode of each logical switch to source node as OVN will only support source node replication. Signed-off-by: Darrell Ball ---

Re: [ovs-dev] [patch_v3] ovn: Add datapaths of interest filtering.

2016-08-09 Thread Darrell Ball
On Thu, Aug 4, 2016 at 3:26 AM, Liran Schour wrote: > "dev" wrote on 03/08/2016 09:09:48 AM: > > > From: Darrell Ball > > To: dlu...@gmail.com, d...@openvswitch.com, b...@ovn.org > > Date: 03/08/2016 09:10 AM > > Subject: [ovs-dev] [patch_v3] ovn: Add d

Re: [ovs-dev] [patch_v8] ovn: Fix receive from vxlan in ovn-controller.

2016-08-10 Thread Darrell Ball
On Wed, Aug 10, 2016 at 4:33 PM, Justin Pettit wrote: > > > On Aug 8, 2016, at 7:20 PM, Darrell Ball wrote: > > > > > > diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c > > index 589b053..43885fd 100644 > > --- a/ovn/controller/physica

Re: [ovs-dev] [patch_v4] ovn: Add local router support (RFC)

2016-06-07 Thread Darrell Ball
On Mon, Jun 6, 2016 at 9:55 PM, Ben Pfaff wrote: > On Tue, May 31, 2016 at 09:37:10AM -0700, Darrell Ball wrote: > > This patch adds local router support. The CMS provides a hint for > > a NB distributed logical router as to which logical switches the > > logical router ser

Re: [ovs-dev] [patch_v1] ovn: Fix receive from vxlan in ovn-controller.

2016-06-07 Thread Darrell Ball
On Tue, Jun 7, 2016 at 8:50 AM, Ben Pfaff wrote: > On Wed, Jun 01, 2016 at 10:34:24AM -0700, Darrell Ball wrote: > > OVN only supports source_node replication and previously vtep > interaction, > > which used service node replication by default for > multicast/broadcas

[ovs-dev] [patch_v2] ovn: Fix receive from vxlan in ovn-controller.

2016-06-08 Thread Darrell Ball
://dillinger.io/ As part of this change ovn-controller-vtep is hard-coded to set the replication mode of each logical switch to source node as OVN will only support source node replication. Signed-off-by: Darrell Ball --- v1->v2: Rebased after recent conflicting commit. Converted some xml comme

[ovs-dev] [patch_v2] ovn: Fix receive from vxlan in ovn-controller.

2016-06-08 Thread Darrell Ball
://dillinger.io/ As part of this change ovn-controller-vtep is hard-coded to set the replication mode of each logical switch to source node as OVN will only support source node replication. Signed-off-by: Darrell Ball --- v1->v2: Rebased after recent conflicting commit. Converted some xml comme

Re: [ovs-dev] [PATCH 4/8] packets: Parse IP address strings with a zero length prefix.

2016-06-09 Thread Darrell Ball
On Thu, Jun 9, 2016 at 12:12 AM, Justin Pettit wrote: > A zero prefix length is used to match any IP address, which is useful > for defining default routes. > > Signed-off-by: Justin Pettit > --- > lib/packets.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a

  1   2   3   >