<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-20 16:51 GMT+08:00 Liu Yuan <span dir="ltr"><<a href="mailto:namei.unix@gmail.com" target="_blank">namei.unix@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Fri, Jun 20, 2014 at 04:21:27PM +0800, Robin Dong wrote:<br>
> From: Robin Dong <<a href="mailto:sanbai@taobao.com">sanbai@taobao.com</a>><br>
><br>
> After following steps:<br>
><br>
>  1. start 6 sheep cluster, every sheep has 4 data path<br>
>  2. unplug 4 data path for one sheep daemon<br>
><br>
> then we will see the sheep daemon which has no data path is also<br>
> in the cluster by using 'dog node list'.But a sheep daemon with<br>
> no data path should be a pure gateway.<br>
><br>
> So we should return EIO after unpluging last disk which will cause<br>
> sheep to leave cluster.<br>
><br>
<br>
</div>sheep will leave cluster by EIO later when it tries to access disks. See<br>
md_handle_eio().<br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Suppose you unplug all the 4 disks for maintainance and then plug back 4 new<br>
disks. We should allow this use case, so this patch looks wrong to me. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
By the way, what is purpose of making a sheep as pure gateway while running?<br></blockquote><div><br></div><div>If all disks corrupt, the sheep daemon on this node will become a pure gateway, after this,</div><div>some one 's wrong operation (such as plug a new disk) will make all nodes coredump.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Yuan<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>--<br>Best Regard<br>Robin Dong
</div></div>