<div dir="ltr"><div><div><div>yes, we need to override all the necessary functions keeping the other function untouched. In other words it is replicate of non-GD2 libs with only changes needed for GD2 functions. <br><br></div>Any other approach to support both GD2 and non-GD2 <br><br></div>Regards,<br></div>Vijay A<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 26, 2018 at 2:07 PM, Nigel Babu <span dir="ltr">&lt;<a href="mailto:nigelb@redhat.com" target="_blank">nigelb@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hang on, when you try to create a file inside this new folder, you can&#39;t override one function, you have to override all the functions in that folder. That&#39;s a lot of code duplication. Is that the best idea forward here?<br><div><div><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 26, 2018 at 12:46 PM Nigel Babu &lt;<a href="mailto:nigelb@redhat.com" target="_blank">nigelb@redhat.com</a>&gt; 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>This seems like an incredibly tricky way to do it. I can certainly see the benefits. I&#39;m a little hesitant to try this approach because it means copying the entire function over for an override and making small changes.</div><div><br></div><div>Either way, the challenge is going to be making sure that changes are in sync for GD2 and non-GD2 tests. Perhaps this is a better approach since you can land changes to both in one commit.<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 12, 2018 at 12:18 PM Vijay Bhaskar Reddy Avuthu &lt;<a href="mailto:vavuthu@redhat.com" target="_blank">vavuthu@redhat.com</a>&gt; 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><div><div><div>I think both cli commands and syntax is changed.<br><br></div>Can we create another directory for GD2 in glusto-tests like below<br><br></div>&quot;glusto-tests/gd2_libs/<wbr>glustolibs/gluster/&quot;<br><br></div>If we need to run the test cases on gd2 setup, then we need to install libraries in gd2_libs on top of usual libs.<br><br><br></div>Regards,<br></div>Vijay A<br><div><div><br><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 11, 2018 at 7:30 PM, Jonathan Holloway <span dir="ltr">&lt;<a href="mailto:jholloway@redhat.com" target="_blank">jholloway@redhat.com</a>&gt;</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>Is the only change in the cli commands the renaming to glustercli, or are some of the commands changing in syntax as well?<br></div><div><br></div>Cheers,<br></div>Jonathan<br></div>
</blockquote></div><br></div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="m_-5542692453338567521m_2439220478063914658gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">nigelb<br></div></div>
</blockquote></div><br clear="all"><br></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr" class="m_-5542692453338567521gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">nigelb<br></div></div></font></span></div></div>
</blockquote></div><br></div>