[sheepdog] [PATCH v3] sheep: remove master node

MORITA Kazutaka morita.kazutaka at lab.ntt.co.jp
Wed Jul 24 09:48:07 CEST 2013


At Wed, 24 Jul 2013 15:38:12 +0800,
Liu Yuan wrote:
> 
> On Wed, Jul 24, 2013 at 04:00:33PM +0900, MORITA Kazutaka wrote:
> > At Tue, 23 Jul 2013 17:23:44 +0800,
> > Kai Zhang wrote:
> > > 
> > > On Jul 23, 2013, at 4:44 PM, MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp> wrote:
> > > 
> > > > Should I include my zookeer patch into this series?  You know, my
> > > > zookeeper patch didn't remove the zk master in either way.  But if you
> > > > want to do your work onto my patch, I'll include my zk patch again
> > > > into the v4 series.
> > > 
> > > If the patch is an separate one and has been done, I would like to see it in v4 : )
> > 
> > Well, the patch was in my v1 series.
> > 
> >   http://lists.wpkg.org/pipermail/sheepdog/2013-July/010804.html
> > 
> > You will try to completely remove a master from the zk driver, right?
> > If the change looks okay to you, I'll add it to v4.  If not sure, I
> > think it's better to keep it out of this series.
> > 
> 
> I think you should keep zk code out of this patch. And write a new patch for it,

Okay.  I'll update v4 after adding some comments to the code.

> then Kai can work on top of it. After all problems are sorted out, I'll apply
> them as a bundle.

Any reason this patch should be pending until Kai finishes his work?
I think this patch doesn't introduce any problem.

Thanks,

Kazutaka



More information about the sheepdog mailing list