Friendly ping

> -----Original Message-----
> From: wangyunjian
> Sent: Saturday, December 19, 2020 7:07 PM
> To: [email protected]
> Cc: [email protected]; [email protected]; Lilijun (Jerry)
> <[email protected]>; xudingke <[email protected]>; chenchanghu
> <[email protected]>; wangyunjian <[email protected]>
> Subject: [ovs-dev] [PATCH] ovs-lib: Fix call exit ovs-vswitchd and 
> ovsdb-server
> failure
> 
> From: Yunjian Wang <[email protected]>
> 
> The ovn-controller need --restart argument. But ovs-vswitchd and
> ovsdb-server don't need argument. This also lead to exit failure.
> 
> Fixes: 7491caa37f3b ("ovn: Modify restart_controller in ovn-ctl to use
> --restart")
> Signed-off-by: Yunjian Wang <[email protected]>
> ---
>  utilities/ovs-lib.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in
> index f7e975674..ab38ece45 100644
> --- a/utilities/ovs-lib.in
> +++ b/utilities/ovs-lib.in
> @@ -273,7 +273,7 @@ stop_daemon () {
>                  case $action in
>                      EXIT)
>                          action "Exiting $1 ($pid)" \
> -                            ${bindir}/ovs-appctl -T 1 -t
> $rundir/$1.$pid.ctl exit $2
> +                            ${bindir}/ovs-appctl -T 1 -t
> $rundir/$1.$pid.ctl exit
>                          # The above command could have resulted in
> delayed
>                          # daemon segfault. And if a monitor is running,
> it
>                          # would restart the daemon giving it a new pid.
> --
> 2.23.0

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to