Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Ben Pfaff
On Thu, May 04, 2017 at 11:14:01AM -0700, Han Zhou wrote: > On Thu, May 4, 2017 at 10:58 AM, Ben Pfaff wrote: > > > > On Thu, May 04, 2017 at 10:34:38AM -0700, Han Zhou wrote: > > > On Thu, May 4, 2017 at 10:05 AM, Ben Pfaff wrote: > > > > > > > > On Wed, May 03, 2017 at 11:43:15PM -0700, Han Zho

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Han Zhou
On Thu, May 4, 2017 at 10:58 AM, Ben Pfaff wrote: > > On Thu, May 04, 2017 at 10:34:38AM -0700, Han Zhou wrote: > > On Thu, May 4, 2017 at 10:05 AM, Ben Pfaff wrote: > > > > > > On Wed, May 03, 2017 at 11:43:15PM -0700, Han Zhou wrote: > > > > On Wed, May 3, 2017 at 4:55 PM, Ben Pfaff wrote: > >

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Ben Pfaff
On Thu, May 04, 2017 at 10:34:38AM -0700, Han Zhou wrote: > On Thu, May 4, 2017 at 10:05 AM, Ben Pfaff wrote: > > > > On Wed, May 03, 2017 at 11:43:15PM -0700, Han Zhou wrote: > > > On Wed, May 3, 2017 at 4:55 PM, Ben Pfaff wrote: > > > > > > > > On Wed, May 03, 2017 at 07:42:46PM -0400, Russell

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Han Zhou
On Thu, May 4, 2017 at 10:05 AM, Ben Pfaff wrote: > > On Wed, May 03, 2017 at 11:43:15PM -0700, Han Zhou wrote: > > On Wed, May 3, 2017 at 4:55 PM, Ben Pfaff wrote: > > > > > > On Wed, May 03, 2017 at 07:42:46PM -0400, Russell Bryant wrote: > > > > On Wed, May 3, 2017 at 11:45 AM, Ben Pfaff wrot

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Ben Pfaff
Yes, I'm in favor of having generic names available. On Thu, May 04, 2017 at 09:49:30AM +0200, Miguel Angel Ajo Pelayo wrote: > Isn't neutron:network_name and neutron:router_name very openstack specific? > > May we introduce more generic names in a way that other systems could > eventually use th

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Ben Pfaff
On Wed, May 03, 2017 at 11:43:15PM -0700, Han Zhou wrote: > On Wed, May 3, 2017 at 4:55 PM, Ben Pfaff wrote: > > > > On Wed, May 03, 2017 at 07:42:46PM -0400, Russell Bryant wrote: > > > On Wed, May 3, 2017 at 11:45 AM, Ben Pfaff wrote: > > > > It's much easier to see what's going on in the south

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-04 Thread Miguel Angel Ajo Pelayo
Isn't neutron:network_name and neutron:router_name very openstack specific? May we introduce more generic names in a way that other systems could eventually use them too? On May 4, 2017 8:43 AM, "Han Zhou" wrote: > On Wed, May 3, 2017 at 4:55 PM, Ben Pfaff wrote: > > > > On Wed, May 03, 2017 a

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-03 Thread Han Zhou
On Wed, May 3, 2017 at 4:55 PM, Ben Pfaff wrote: > > On Wed, May 03, 2017 at 07:42:46PM -0400, Russell Bryant wrote: > > On Wed, May 3, 2017 at 11:45 AM, Ben Pfaff wrote: > > > It's much easier to see what's going on in the southbound database if > > > human-friendly names are available. > > > >

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-03 Thread Ben Pfaff
On Wed, May 03, 2017 at 07:42:46PM -0400, Russell Bryant wrote: > On Wed, May 3, 2017 at 11:45 AM, Ben Pfaff wrote: > > It's much easier to see what's going on in the southbound database if > > human-friendly names are available. > > > > Really it's too bad that we didn't put the human-friendly na

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-03 Thread Russell Bryant
On Wed, May 3, 2017 at 11:45 AM, Ben Pfaff wrote: > It's much easier to see what's going on in the southbound database if > human-friendly names are available. > > Really it's too bad that we didn't put the human-friendly name in "name" > and the UUID in something like "external_ids:neutron-uuid",

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-03 Thread Ben Pfaff
On Wed, May 03, 2017 at 02:59:15PM -0700, Andy Zhou wrote: > On Wed, May 3, 2017 at 8:45 AM, Ben Pfaff wrote: > > It's much easier to see what's going on in the southbound database if > > human-friendly names are available. > > > > Really it's too bad that we didn't put the human-friendly name in

Re: [ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-03 Thread Andy Zhou
On Wed, May 3, 2017 at 8:45 AM, Ben Pfaff wrote: > It's much easier to see what's going on in the southbound database if > human-friendly names are available. > > Really it's too bad that we didn't put the human-friendly name in "name" > and the UUID in something like "external_ids:neutron-uuid",

[ovs-dev] [PATCH v2 02/13] ovn-northd: Propagate Neutron datapath names to southbound database.

2017-05-03 Thread Ben Pfaff
It's much easier to see what's going on in the southbound database if human-friendly names are available. Really it's too bad that we didn't put the human-friendly name in "name" and the UUID in something like "external_ids:neutron-uuid", but it'll take more coordination to change that at this poi