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

Ben Pfaff blp at ovn.org
Wed Dec 16 06:59:58 UTC 2015


On Tue, Dec 15, 2015 at 09:52:02AM -0800, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty <guru at 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.



More information about the dev mailing list