<div dir="ltr">Distributors will surely want to build all checkers to have the docs and hwtable in sync with the shipped checkers, and cover the widest range of setups.<div><br></div><div>I'd rather drop this patch or, if it proves to be a problem, merge a variant with the rbd checker build by default and not-buildable by explicit define.</div><div><br></div><div>But thanks,</div><div>Christophe</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 9, 2016 at 8:26 PM, Mike Christie <span dir="ltr"><<a href="mailto:mchristi@redhat.com" target="_blank">mchristi@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08/09/2016 10:36 AM, Christophe Varoqui wrote:<br>
> Merged.<br>
><br>
<br>
Thanks.<br>
<span class=""><br>
> I think it is worth advertizing here that the build now requires rados.h.<br>
><br>
<br>
</span>Sorry about adding the silly dependency. How about the attached patch to<br>
make the rbd checker only be built if the user requests it. In the<br>
documentation for my use, I will describe how to build it. This way<br>
majority of users will not be bothered.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> Thanks.<br>
><br>
> On Mon, Aug 8, 2016 at 2:01 PM, Mike Christie <<a href="mailto:mchristi@redhat.com">mchristi@redhat.com</a>> wrote:<br>
><br>
>> The following patches made over Christophe's tree today,<br>
>> add Ceph rbd support for handling blacklisted devices.<br>
>><br>
>><br>
>> This is not general support for rbd and multipath. There is<br>
>> no boot/root support and it does not support features like<br>
>> multibus. My use is for HA configurations, specifically for exporting<br>
>> rbd images through multiple LIO instances. In this case, we have one<br>
>> rbd instance that has the ceph rbd exclusive lock and it can send<br>
>> WRITE requests. If that host becomes unreachable, then another host<br>
>> will grab the lock, and blacklist the original host to prevent it from<br>
>> sending stale IO (when blacklisted IO will be failed by the OSD).<br>
>><br>
>> To recover from that type of scenario, this patchset adds a repair()<br>
>> callout to the checker. If the path is in the PATH_DOWN state this<br>
>> callout can be used to fix it up. For my case, I am remapping<br>
>> the device to flush stale IO and cleanup the old lock,<br>
>> and then unblacklisting the path, so it can be used again.<br>
>><br>
>> Changes since v1:<br>
>> - Drop ID_UID use and implemented sysfs getuid support.<br>
>> - Drop settings that were defaults and follow template.<br>
>> - Fix ceph auth/user.<br>
>><br>
>><br>
><br>
><br>
><br>
</div></div><span class="HOEnZb"><font color="#888888">> --<br>
> dm-devel mailing list<br>
> <a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
> <a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/dm-devel</a><br>
><br>
<br>
</font></span></blockquote></div><br></div>