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: > > > > > > > > This patch series updates the vtep schema,

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

2016-05-08 Thread Ben Pfaff
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: > > > > > > This patch series updates the vtep schema, vtep-ctl commands and vtep > > > > It's not really a series. E

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

2016-05-07 Thread Darrell Ball
> > > +responsible for doing their own replication for locally attached VMs in >> > +both modes. Service node mode is the default and was the only option >> for >> > +prior versions of the VTEP schema. Service node replication mode is >> >> I would drop reference to the prior versions of the VTEP

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

2016-05-06 Thread Justin Pettit
> 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 "service_node" is implied. > > I'm not sure that "return value of NULL" is exactly a correct description. > Perhaps "an empty respon

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 > history once its committed, it wo

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

2016-05-06 Thread Justin Pettit
> 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 history once its committed, it wouldn't show as a series. > diff --git a/vtep/README.ovs-vtep.md

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

2016-05-04 Thread Anupam Chanda
Looks good. Acked-by: Anupam Chanda Thanks, Anupam ​ From: Darrell Ball mailto:dlu...@gmail.com>> Date: Wed, May 4, 2016 at 12:03 PM Subject: [patch_v9] vtep: add source node replication support. To: dlu...@gmail.com, bda...@vmware.com,

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

2016-05-04 Thread Darrell Ball
This patch series updates the vtep schema, vtep-ctl commands and vtep simulator to support source node replication in addition to service node replication per logical switch. The default replication mode is service node as that was the only mode previously supported. Source node replication mode

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

2016-05-04 Thread Darrell Ball
This patch series updates the vtep schema to support source node replication in addition to service node replication per logical switch. The default replication mode is service_node as that was the only mode previously supported. New vtep-ctl commands are added to set/reset/get the replication mo