[ovs-dev] [PATCH v2 OVN] ovn-ctl: Fix stucked while do cluster db init

[Han Zhou]

On Mon, Mar 16, 2020 at 8:16 PM Tao YunXiang <
taoyunxiang at cmss.chinamobile.com> wrote:
>
> For cluster mode, only the first server, which is the leader, in a cluster
> should do init. But, sometimes, the role of the first server would
transfer
> leadership, so we should add --no-leader-only in case the ovn-northd
stucked.
>
>
> v1: ovn-ctl: Add "--no-leader-only" while cluster db init
>
> Author: Tao YunXiang <taoyunxiang at cmss.chinamobile.com>
> Co-authored-by: Liu Chang <liuchang at cmss.chinamobile.com>
> Co-authored-by: Rong Yin <rongyin at cmss.chinamobile.com>
> Signed-off-by: Tao YunXiang <taoyunxiang at cmss.chinamobile.com>
> Signed-off-by: Liu Chang <liuchang at cmss.chinamobile.com>
> Signed-off-by: Rong Yin <rongyin at cmss.chinamobile.com>
>
> ---
>  utilities/ovn-ctl | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl
> index 2a337ae27..8afe68a0a 100755
> --- a/utilities/ovn-ctl
> +++ b/utilities/ovn-ctl
> @@ -290,10 +290,9 @@ $cluster_remote_port
>
>      "$@" "$file"
>
> -    # Initialize the database if it's running standalone,
> -    # active-passive, or is the first server in a cluster.
> +    # Initialize the database if it's NOT joining a cluster.
>      if test -z "$cluster_remote_addr"; then
> -        $(echo ovn-${db}ctl | tr _ -) init
> +        $(echo ovn-${db}ctl | tr _ -) --no-leader-only init
>      fi
>
>      if test $mode = cluster; then
> --
> 2.17.1
>
>
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Thanks for the revision. I applied to master after removing the below line
in commit message:

v1: ovn-ctl: Add "--no-leader-only" while cluster db init

(Usually, notes about different versions of patches are added after the
line "---" in the patch, so that it will be ignored by git automatically.)