[ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-15 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- ovn/utilities/ovn-ctl |6 ++ 1 file changed, 6 insertions(+) diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl index 3e2ccf9..b171934 100755 --- a/ovn/utilities/ovn-ctl +++ b/ovn/utilities/ovn-ctl @@ -225,6 +225,12 @@ case $command in re

Re: [ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-15 Thread Russell Bryant
On 12/15/2015 12:52 PM, Gurucharan Shetty wrote: > Signed-off-by: Gurucharan Shetty > --- > ovn/utilities/ovn-ctl |6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl > index 3e2ccf9..b171934 100755 > --- a/ovn/utilities/ovn-ctl > +++ b/

Re: [ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-15 Thread Ben Pfaff
On Tue, Dec 15, 2015 at 09:52:02AM -0800, Gurucharan Shetty wrote: > Signed-off-by: Gurucharan Shetty > --- > ovn/utilities/ovn-ctl |6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl > index 3e2ccf9..b171934 100755 > --- a/ovn/utilitie

Re: [ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-15 Thread Ben Pfaff
On Tue, Dec 15, 2015 at 10:59:58PM -0800, Ben Pfaff wrote: > On Tue, Dec 15, 2015 at 09:52:02AM -0800, Gurucharan Shetty wrote: > > Signed-off-by: Gurucharan Shetty > > --- > > ovn/utilities/ovn-ctl |6 ++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/ovn/utilities/ovn-ctl b/o

Re: [ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-16 Thread Russell Bryant
On 12/16/2015 02:00 AM, Ben Pfaff wrote: > On Tue, Dec 15, 2015 at 10:59:58PM -0800, Ben Pfaff wrote: >> On Tue, Dec 15, 2015 at 09:52:02AM -0800, Gurucharan Shetty wrote: >>> Signed-off-by: Gurucharan Shetty >>> --- >>> ovn/utilities/ovn-ctl |6 ++ >>> 1 file changed, 6 insertions(+) >>>

Re: [ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-16 Thread Guru Shetty
> > > Do you think it's worth breaking out individual statuses this way? > "ovs-ctl status" shows all OVS daemons' status; maybe "ovn-ctl status" > should follow the same pattern. > The reason I chose to break it was because I was splitting ovn-northd and ovn-controller into separate packages with

Re: [ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

2015-12-23 Thread Ben Pfaff
On Wed, Dec 16, 2015 at 07:23:13AM -0800, Guru Shetty wrote: > > Do you think it's worth breaking out individual statuses this way? > > "ovs-ctl status" shows all OVS daemons' status; maybe "ovn-ctl status" > > should follow the same pattern. > > > The reason I chose to break it was because I was s