<div dir="ltr">All I know is pump translator is written long back to support 'replace-brick' without data-loss in distributed setup too. But we figured out pump is not maintainable along the way, and said you can't do a replace-brick in distributed volume type.<div><br></div><div><a href="https://github.com/gluster/glusterfs/commit/acdeed002d30209e0a058c2df0346d4f16c08994" target="_blank">https://github.com/gluster/<wbr>glusterfs/commit/<wbr>acdeed002d30209e0a058c2df0346d<wbr>4f16c08994</a><br></div><div><br></div><div>I don't see that its used now, so looks like we can remove this test. </div><div><br></div><div>-Amar</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 8, 2017 at 10:02 AM, Atin Mukherjee <span dir="ltr"><<a href="mailto:amukherj@redhat.com" target="_blank">amukherj@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Pranith,<br><br></div>I see you're the author of the test in $Subj. Now while I was working on a patch <a href="https://review.gluster.org/#/c/18226/" target="_blank">https://review.gluster.org/#/<wbr>c/18226/</a> to disallow replace brick operations on dist only volumes the patch failed the regression on this test as the test actually uses replace brick on a distribute only volume which IMO is wrong as then this would always end up in to data loss situation. I'd need some context here to understand the expectation of this test before doing any modifications.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div></div><div>~Atin<br></div></font></span></div>
<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><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Amar Tumballi (amarts)<br></div></div></div></div></div>
</div>