Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread Numan Siddique
On Wed, Nov 8, 2017 at 9:28 AM, Ben Pfaff wrote: > On Wed, Nov 08, 2017 at 08:48:25AM +0530, Numan Siddique wrote: > > On Wed, Nov 8, 2017 at 8:39 AM, Ben Pfaff wrote: > > > > > On Tue, Nov 07, 2017 at 09:01:27PM +0530, nusid...@redhat.com wrote: > > > > From: Numan Siddique > > > > > > > > ovn

Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread Ben Pfaff
On Wed, Nov 08, 2017 at 08:48:25AM +0530, Numan Siddique wrote: > On Wed, Nov 8, 2017 at 8:39 AM, Ben Pfaff wrote: > > > On Tue, Nov 07, 2017 at 09:01:27PM +0530, nusid...@redhat.com wrote: > > > From: Numan Siddique > > > > > > ovn-ctl starts OVN DB servers with the options "--unixctl=ovnnb_db"

Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread Numan Siddique
On Wed, Nov 8, 2017 at 8:39 AM, Ben Pfaff wrote: > On Tue, Nov 07, 2017 at 09:01:27PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique > > > > ovn-ctl starts OVN DB servers with the options "--unixctl=ovnnb_db" > > and "--unixctl=ovnsb_db". When "ovs-appctl -t ovnnb_db" is run the > >

Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread Ben Pfaff
On Tue, Nov 07, 2017 at 09:01:27PM +0530, nusid...@redhat.com wrote: > From: Numan Siddique > > ovn-ctl starts OVN DB servers with the options "--unixctl=ovnnb_db" > and "--unixctl=ovnsb_db". When "ovs-appctl -t ovnnb_db" is run the > below error is seen > > 2017-11-07T10:18:17Z|1|unixctl|WA

Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread Numan Siddique
Thanks Mark for the comments. On Wed, Nov 8, 2017 at 2:30 AM, Mark Michelson wrote: > I'm not sure what to think of this one. This fixes a problem when using > ovn-ctl to start the databases. But I'm concerned about this change causing > breakage for deployments that don't use ovn-ctl. > When

Re: [ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread Mark Michelson
I'm not sure what to think of this one. This fixes a problem when using ovn-ctl to start the databases. But I'm concerned about this change causing breakage for deployments that don't use ovn-ctl. So I guess the question is, does anyone ever actually start the OVN databases with something other tha

[ovs-dev] [PATCH 2/2] ovs-appctl: Allow ovnnb_db and ovnsb_db as targets

2017-11-07 Thread nusiddiq
From: Numan Siddique ovn-ctl starts OVN DB servers with the options "--unixctl=ovnnb_db" and "--unixctl=ovnsb_db". When "ovs-appctl -t ovnnb_db" is run the below error is seen 2017-11-07T10:18:17Z|1|unixctl|WARN|failed to connect to /var/run/openvswitch/ovnnb_db.29536.ctl. This patch allows