<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 7 January 2018 at 18:54, Jeff Darcy <span dir="ltr"><<a href="mailto:jeff@pl.atyp.us" target="_blank">jeff@pl.atyp.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There's no explanation, or reference to one, in the commit message. In the comments, there's a claim that seems a bit exaggerated.<br>
<br>
> This is causing almost all the regressions to fail. durbaility-off.t is the most affected test.<br></blockquote><div><br></div><div>This patch does seem to be the cause. Running this test in a loop on my local system with the patch caused it to fail several times (it is an intermittent failure). The test passed every time after I tried the same after reverting the patch. When it fails, it is because the mount process does not connect to all bricks.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This patch was merged on December 13. Regressions have passed many times since then. If almost all regressions have started failing recently, I suggest we look for a more recent cause. For example, if this was collateral damage from debugging the dict-change issue, then the patch should be reinstated (which I see has not been done). Alternatively, is the above supposed to mean that this patch has been observed to cause *occasional* failures in many other tests? If so, which tests and when? There's no way to search for these in Gerrit or Jenkins. If specific logs or core-dump analyses point toward this conclusion and the subsequent action, then it would be very helpful for those to be brought forward so we can debug the underlying problem. That's likely to be hard enough without trying to do it blind.<br>
______________________________<wbr>_________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org">Gluster-devel@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/gluster-devel</a><br>
</blockquote></div><br></div></div>