<div><br></div><div><br><div class="gmail_quote"><div dir="ltr">On Fri, 29 Jun 2018 at 21:35, Poornima Gurusiddaiah <<a href="mailto:pgurusid@redhat.com">pgurusid@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 29, 2018, 5:54 PM Mohit Agrawal <<a href="mailto:moagrawa@redhat.com" target="_blank">moagrawa@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Poornima,</div><div><br></div><div>It seems the test case(tests/bugs/core/bug-1432542-mpx-restart-crash.t) is crashing because in quota xlator rpc-cleanup code is not perfectly handled in brick multiplexing scenario. Though I did not see this issue from a long time after done some changes in quota xlator code but now it seems on your patch it is consistently reproducible.</div><div><br></div><div> For this build to test your code you can mark it as a bad test and try to run a regression.I will check how we can resolve the same.</div></div></div></blockquote></div></div></div><div dir="auto"><div dir="auto">I have done that already, and all the regression tests pass if I mark this one bad. So it's only this the regression is blocked on. Is it possible to fix it in the near future? Or mark the test bad until then?</div></div></blockquote><div dir="auto"><br></div><div dir="auto">We shouldn’t be marking a test bad to get a patch in. That’s totally unacceptable. We should try to fix this test first. Btw, I see for most of the patches the regression passes, do what’s in for Poornima’s patch which doesn’t get through with this test? Is it just intermittent?</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div dir="auto"></div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">Poornima</div></div><div dir="auto"><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Regards</div><div>Mohit Agrawal</div></div>
_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" rel="noreferrer" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer noreferrer" target="_blank">http://lists.gluster.org/mailman/listinfo/gluster-devel</a></blockquote></div></div></div>
_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://lists.gluster.org/mailman/listinfo/gluster-devel</a></blockquote></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">- Atin (atinm)</div>