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 <g...@ovn.org> >>> --- >>> 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 >>> restart_controller) >>> restart_controller >>> ;; >>> + status_northd) >>> + daemon_status ovn-northd || exit 1 >>> + ;; >>> + status_controller) >>> + daemon_status ovn-controller || exit 1 >>> + ;; >> >> 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. > > Oh, but in case you disagree, I don't feel strongly, so have an ack if > you want: > Acked-by: Ben Pfaff <b...@ovn.org>
It seems reasonable that you might want to check the status of only ovn-controller on a hypervisor, for example. We could have a third "status" arg that checks all though. Acked-by: Russell Bryant <russ...@ovn.org> -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
