<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 18 March 2016 at 02:47, 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><div style="font-family:times new roman,new york,times,serif;font-size:12pt;color:#000000"><div><br></div><div><br></div><hr><blockquote style="border-left:2px solid #1010ff;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><b>From: </b>&quot;M S Vishwanath Bhat&quot; &lt;<a href="mailto:msvbhat@gmail.com" target="_blank">msvbhat@gmail.com</a>&gt;<br><b>To: </b>&quot;Niels de Vos&quot; &lt;<a href="mailto:ndevos@redhat.com" target="_blank">ndevos@redhat.com</a>&gt;<br><b>Cc: </b>&quot;Gluster Devel&quot; &lt;<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a>&gt;<br><b>Sent: </b>Thursday, March 17, 2016 8:18:23 AM<br><b>Subject: </b>Re: [Gluster-devel] Location of distaf tests<div><div class="h5"><br><div><br></div><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 17 March 2016 at 10:50, Niels de Vos <span dir="ltr">&lt;<a href="mailto:ndevos@redhat.com" target="_blank">ndevos@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><div>On Wed, Mar 09, 2016 at 08:26:44PM +0530, M S Vishwanath Bhat wrote:<br> &gt; On 9 March 2016 at 19:39, Kaushal M &lt;<a href="mailto:kshlmster@gmail.com" target="_blank">kshlmster@gmail.com</a>&gt; wrote:<br> &gt;<br> &gt; &gt; On Wed, Mar 9, 2016 at 7:02 PM, M S Vishwanath Bhat &lt;<a href="mailto:msvbhat@gmail.com" target="_blank">msvbhat@gmail.com</a>&gt;<br> &gt; &gt; wrote:<br> &gt; &gt; &gt; Hi,<br> &gt; &gt; &gt;<br> &gt; &gt; &gt; When we were discussing about the readiness of distaf for upstream test<br> &gt; &gt; &gt; automation, this question came up, That we should have a process or<br> &gt; &gt; workflow<br> &gt; &gt; &gt; for proposing, reviewing and including the tests somewhere.<br> &gt; &gt; &gt;<br> &gt; &gt; &gt; Right now the tests are part of distaf repository<br> &gt; &gt; &gt; (<a href="http://github.com/gluster/distaf" rel="noreferrer" target="_blank">github.com/gluster/distaf</a>) itself. And contributing to distaf is by<br> &gt; &gt; sending<br> &gt; &gt; &gt; a PR. But we want this to be included in gerrit so that review and<br> &gt; &gt; &gt; contributing process becomes much easier. But the question still<br> &gt; &gt; remains...<br> &gt; &gt; &gt; where? Right now I can think of below options.<br> &gt; &gt; &gt;<br> &gt; &gt; &gt; * Use the same distaf repo in github for tests as well.<br> &gt; &gt; &gt; * Create a separate repo distaf_gluster_tests (or something similar) and<br> &gt; &gt; &gt; have all the tests there.<br> &gt; &gt; &gt; * Or have a tests/distaf/ directory inside glusterfs repository. And this<br> &gt; &gt; &gt; tests can be bundled in a rpm and distributed. This directory will have<br> &gt; &gt; both<br> &gt; &gt; &gt; the test cases and related library functions.<br> &gt; &gt;<br> &gt; &gt; I prefer this approach. It makes it easier for developers to submit<br> &gt; &gt; tests along with their changes, as is the case with our regression<br> &gt; &gt; tests now.<br> &gt; &gt;<br> &gt; &gt; By library functions, I&#39;m assuming you mean helper libraries related<br> &gt; &gt; to gluster, which will be used in the tests which will be written.<br> &gt; &gt;<br> &gt;<br> &gt; Yes, I mean helper functions which are related to gluster. The framework<br> &gt; itself will be  made a python package. At least that&#39;s the plan.<br><div><br></div></div></div>+1 for the tests in tests/distaf/ . I think I would like the libs to be<br> part of the main distaf project. That would make it easier to run distaf<br> for other projects that integrate with Gluster (like Samba,<br> NFS-Ganesha and the like). The upstream projects can that use distaf for<br> their integration with Gluster if they choose so.<br></blockquote><div><br></div><div>My concern with having libs in main distaf project is that, libs and test cases are very much tied together. So when someone writes a testcase, they write and/or modify the related gluster libs along with it. And the one needs to go back and forth between the testcase and gluster libs while writing and debugging the case. If they are in separate repo/package, it makes it bit difficult (not impossible) for test case writer.</div><div><br></div><div>If everyone is of the same opinion that gluster libs should be part of main distaf repo, I can still do it when I package it next week.</div></div></div></div></div></div></blockquote><div><br></div><div>I&#39;m now thinking we should go ahead and keep it three distinct projects:<br></div><div>- Gluster: testcase--and supporting functions that are not a good fit for the libs<br></div><div>- distaf-libs-gluster: core libarary and utility functions</div><div>- distaf: framework only<br></div></div></div></blockquote><div><br></div><div>But that (having more than one repo) kind of  defeats the purpose of developers having to concern themselves with only single repo (glusterfs.git). Since I couldn&#39;t discuss this in weekly gluster meeting in #gluster-meeting, let me propose my solution here.</div><div><br></div><div>Library functions and test cases both go into a subdirectory inside glusterfs.git. This helps for devs and people writing test case for gluster. And we find a way to sync the libraries to distaf.git, so that each time a package is made, it is available for integrating projects to make use of. I know this is not very elegant way of dealing with the problem, but I couldn&#39;t think of anything which would fit all requirements.</div><div><br></div><div>Jonathan, Niels, Kaushal, Raghavendra Talur - Your thoughts?</div><div><br></div><div>Best Regards,</div><div>Vishwanath</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:times new roman,new york,times,serif;font-size:12pt;color:#000000"><div></div><div><br></div><div>Here&#39;s some of my reasoning...<br></div><div>- We&#39;re already talking about not putting the libs in the Gluster repo as an option, so from a convenience perspective it&#39;s not as relevant which repo is being used (distaf or distaf-gluster-libs).<br></div><div>- And if we&#39;re already talking about the possibility of a refactor to split the libs out down the road, it&#39;s not much different to talk refactor to roll the libs into the distaf project should they become a problem to maintain.<br></div><div><div>- With that said, if looking to have distaf adopted by a wider non-Gluster audience, we&#39;ll want to split the libs out down the road anyway. It would also lay the foundation for others (distaf-libs-&lt;your-project-here&gt;).<br></div><div><div>- The Maintainer/Reviewer/Contributor demographic isn&#39;t necessarily the same across all three and the Maintainer/Reviewer will most likely play a larger role initially as we build out the libs and figure out what goes where, what works, etc.</div>        I&#39;m just thinking it would be easier to manage that separately in the beginning.</div></div><div>- If we are including the libs in the distaf project, how will the libs be consumed? Roll them in the distaf package? Separate package? Git clone and setup.py?<br></div><div>        I like the concept of separate distaf and distaf-libs-gluster packages that can be versioned and maintained separately. It keeps them flexible and clean/clear of each other, and any framework changes that are required by a library change can be managed through the packaging.<br></div><div>        We can still do that if the libs are in the same repo, but in my opinion it&#39;s not as clean and the value add in having them combined is minimal.<br></div><div>- It shouldn&#39;t take that long to create the new project and may save us some inevitable headache down the road.<br></div><div><br></div><div>Either way, I agree with Niels that the sooner the better--so if a separate repo for distaf-libs-gluster would create an unacceptable delay, rolling libs into the distaf project works for me.<br></div><div><br></div><div>Cheers,<br></div><div>Jonathan<br></div><div><div class="h5"><blockquote style="border-left:2px solid #1010ff;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Best Regards,</div><div>Vishwanath</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><br> &gt; &gt; I&#39;m also in favor of including them here as well. This will help keep<br> &gt; &gt; DiSTAF free of an Gluster specific cruft and allow it to be (possibly)<br> &gt; &gt; reusable by others.<br> &gt; &gt;<br> &gt;<br> &gt; The recent changes makes it specific to gluster, but very easy to make it<br> &gt; generic.<br> <br> </span>Starting Gluster specific is fine, later on when things are in use and<br> working the way we need it should be trivial to make it more generic.<br> Just keep a generic way for adding new functionalities for now, and file<br> issues in GitHub for the parts that need to get refactored.<br><div><br></div>The sooner we can start using distaf and gain (more) real world<br> experience the better.<br><div><br></div>Thanks,<br> Niels<br><div><div><br><div><br></div>&gt; Best Regards,<br> &gt; Vishwanath<br> &gt;<br> &gt;<br> &gt; &gt;<br> &gt; &gt; &gt;<br> &gt; &gt; &gt; Please let us know what your preferred option is. If you have any other<br> &gt; &gt; &gt; ideas, please let us know as well.<br> &gt; &gt; &gt;<br> &gt; &gt; &gt; Best Regards,<br> &gt; &gt; &gt; Vishwanath<br> &gt; &gt; &gt;<br> &gt; &gt; &gt;<br> &gt; &gt; &gt; _______________________________________________<br> &gt; &gt; &gt; Gluster-devel mailing list<br> &gt; &gt; &gt; <a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br> &gt; &gt; &gt; <a href="http://www.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://www.gluster.org/mailman/listinfo/gluster-devel</a><br> &gt; &gt;<br><div><br></div>&gt; _______________________________________________<br> &gt; Gluster-devel mailing list<br> &gt; <a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br> &gt; <a href="http://www.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://www.gluster.org/mailman/listinfo/gluster-devel</a><br><div><br></div></div></div></blockquote></div><br></div></div><br>_______________________________________________<br>Gluster-devel mailing list<br><a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br><a href="http://www.gluster.org/mailman/listinfo/gluster-devel" target="_blank">http://www.gluster.org/mailman/listinfo/gluster-devel</a></blockquote><div><br></div></div></div></div></div></blockquote></div><br></div></div>