[sheepdog] [PATCH] zk: delete zk node before join cluster
MORITA Kazutaka
morita.kazutaka at lab.ntt.co.jp
Fri May 31 03:48:28 CEST 2013
At Fri, 31 May 2013 09:44:10 +0800,
Kai Zhang wrote:
>
>
> On May 30, 2013, at 11:09 PM, Liu Yuan <namei.unix at gmail.com> wrote:
>
> > On 05/30/2013 03:20 PM, Liu Yuan wrote:
> >> On 05/30/2013 02:37 PM, Kai Zhang wrote:
> >>> Delete zk node before join cluster and return -1 when fail.
> >>
> >> Applied, thanks.
> >>
> >
> > Oops, this patch breaks
> >
> > /* For concurrent nodes setup, we allow only one to continue */
> > while (zk_member_empty() && zk_master_create() != ZOK)
> > ;/* wait */
> >
> > I guess this was why in the old code I chose to panic(). Now when I
> > start 3 node concurrently, I'll end up with 3 nodes are all master.
> > Unless we find a solution, this patch will be reverted.
>
> However, current implementation of master election also has problem.
>
> I will rewrite this by using sequential node which is a standard way of implementing master election.
Can you wait with the patch? I have a patch set which will completely
remove a master from the cluster drivers. I think of sending them
after I release the 0.6.0 release.
Thanks,
Kazutaka
More information about the sheepdog
mailing list