[ovs-dev] [PATCH v2] ovn: Add ovn-bridge-mappings to Chassis external_ids.

2016-03-28 Thread Russell Bryant
Publish ovn-controller's local bridge mappings configuration in the external_ids column of the Chassis table. Having this information available for reading is useful to applications integrating with OVN. Signed-off-by: Russell Bryant --- ovn/controller/chassis.c | 24 o

Re: [ovs-dev] [PATCH v2] ovn: Add ovn-bridge-mappings to Chassis external_ids.

2016-03-29 Thread Ryan Moats
This makes sense to me, though before I ack it, I'm wondering if a test case would be a useful addition? Ryan "dev" wrote on 03/28/2016 05:44:49 PM: > From: Russell Bryant > To: dev@openvswitch.org > Date: 03/28/2016 05:45 PM > Subject: [ovs-dev] [PATCH v2] ovn:

Re: [ovs-dev] [PATCH v2] ovn: Add ovn-bridge-mappings to Chassis external_ids.

2016-03-29 Thread Russell Bryant
On Tue, Mar 29, 2016 at 10:43 AM, Ryan Moats wrote: > This makes sense to me, though before I ack it, I'm wondering if a test > case would be a useful addition? > That's a fair request. I have no good excuse for not adding a test case for this. Thanks, -- Russell Bryant __

Re: [ovs-dev] [PATCH v2] ovn: Add ovn-bridge-mappings to Chassis external_ids.

2016-03-29 Thread Ben Pfaff
On Mon, Mar 28, 2016 at 06:44:49PM -0400, Russell Bryant wrote: > Publish ovn-controller's local bridge mappings configuration > in the external_ids column of the Chassis table. Having this > information available for reading is useful to applications > integrating with OVN. > > Signed-off-by: Ru

Re: [ovs-dev] [PATCH v2] ovn: Add ovn-bridge-mappings to Chassis external_ids.

2016-03-29 Thread Russell Bryant
On Tue, Mar 29, 2016 at 11:43 AM, Ben Pfaff wrote: > On Mon, Mar 28, 2016 at 06:44:49PM -0400, Russell Bryant wrote: > > Publish ovn-controller's local bridge mappings configuration > > in the external_ids column of the Chassis table. Having this > > information available for reading is useful t