<div dir="ltr"><div dir="ltr"><br><div>Hi Shankarsan,</div><div><br></div><div>Replies inline.</div><div><br></div><div>Thanks</div><div>Saju</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 25, 2021 at 6:51 PM sankarshan <<a href="mailto:sankarshan@kadalu.io" target="_blank">sankarshan@kadalu.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Ayush - thank you for hosting what is your first Gluster community<br>
meeting! It was an excellent effort at keeping the conversation moving<br>
along.<br>
<br>
Some additional comments in-line.<br>
<br>
On Tue, 25 May 2021 at 17:52, Ayush Ujjwal <<a href="mailto:aujjwal@redhat.com" target="_blank">aujjwal@redhat.com</a>> wrote:<br>
><br>
> # Gluster Community Meeting - 25/05/2021<br>
<br>
[snip]<br>
<br>
> * Project metrics:<br>
><br>
> | Metrics | Value |<br>
> | ------------------------- | -------- |<br>
> |[Coverity](<a href="https://scan.coverity.com/projects/gluster-glusterfs" rel="noreferrer" target="_blank">https://scan.coverity.com/projects/gluster-glusterfs</a>) | 38 |<br>
> |[Clang Scan](<a href="https://build.gluster.org/job/clang-scan/lastBuild/" rel="noreferrer" target="_blank">https://build.gluster.org/job/clang-scan/lastBuild/</a>) | 89 |<br>
> |[Test coverage](<a href="https://build.gluster.org/job/line-coverage/lastCompletedBuild/Line_20Coverage_20Report/)%7C" rel="noreferrer" target="_blank">https://build.gluster.org/job/line-coverage/lastCompletedBuild/Line_20Coverage_20Report/)|</a> 70.9 |<br>
> |[Gluster User Queries in last 14 days](<a href="https://lists.gluster.org/pipermail/gluster-users/2021-May/thread.html#start" rel="noreferrer" target="_blank">https://lists.gluster.org/pipermail/gluster-users/2021-May/thread.html#start</a>) | 27 |<br>
> |[Total Github issues](<a href="https://github.com/gluster/glusterfs/issues" rel="noreferrer" target="_blank">https://github.com/gluster/glusterfs/issues</a>) | 315 |<br>
><br>
<br>
As brought up at the meeting - it might be useful to discuss the trend<br>
of these values and from there deduce if these are in the right<br>
direction. The values in isolation do not communicate enough data to<br>
determine whether there are opportunities to improve. At a certain<br>
point in time in early 2020 there was intense focus on test coverage.<br>
I am not sure if that has resulted in actual better coverage or just<br>
spreading butter on toast.<br></blockquote><div> </div><div>Please follow the gluster-devel mailing list, there is a email sent out every Monday</div><div>With subject: "Gluster Code Metrics Weekly Report", which gives weekly values </div><div>and a trend graph and links to the respective jobs.</div><div>On the test coverage: Yes we need more participation from community.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
><br>
> * Any release updates?<br>
> * None<br>
><br>
> * Blocker issues across the project?<br>
> * It looks like the lock-recovery changes introduced with <a href="https://review.gluster.org/#/c/glusterfs/+/22712/" rel="noreferrer" target="_blank">https://review.gluster.org/#/c/glusterfs/+/22712/</a> has issues. We already fixed <a href="https://github.com/gluster/glusterfs/pull/2456" rel="noreferrer" target="_blank">https://github.com/gluster/glusterfs/pull/2456</a> and <a href="https://github.com/gluster/glusterfs/issues/2337" rel="noreferrer" target="_blank">https://github.com/gluster/glusterfs/issues/2337</a> but looks like the code is buggy. Need someone to take a look at the difference between posix-locks and client xlators in how the locks are maintained to fix the issue completely.<br>
><br>
<br>
As a project it is necessary to do right by our community - this means<br>
that the impact of the issue and remedy/workaround should be<br>
immediately shared widely enough to ensure that this is not missed.<br>
Since the issues are tagged 'blocker' I am guessing that these meet<br>
the somewhat established criteria of a blocker issue and would need an<br>
enhanced level of attention. Have the issues been triaged and<br>
developers assigned?<br></blockquote><div>The above doesnt look to be a blocker issue as this happens on particular locking pattern. Discussed and confirmed this with Pranit yesterday. The fix for this will go in the next minor releases happening in June. We do highlight issues in the release notes; this is not a showstopper one.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
><br>
> * Notable thread form mailing list<br>
> * Not exactly from mailing list. Slack user pinged me and asked me if it is possible to let the users know of any known issues in the latest releases so that they can make a decision about which version to use. For example: 9.0 and 9.1 had protocol issue.<br>
> * Along the same lines, I wanted to ask one more question. Should we release beta-releases for major releases so that we get feedback about any issues that happen in their particular environment to address the issues even before the stable releases are made?<br>
><br>
><br>
<br>
I've offered to look at the criteria which defines a 'beta' and check<br>
how it aligns with a release schedule. The history of 'beta' releases<br>
of storage software (as compared to say a browser) is that we have<br>
often received no uptake. There are many reasons for this - but one<br>
key aspect is that it is additional work being asked from the<br>
community. If the 'beta' is reasonably well described perhaps the<br>
accrued value from this testing cycle would be better understood.<br>
________<br></blockquote><div>The key aspect statement: "additional work being asked from the</div>
community" itself is worrying. The community should help and serve each other.<div>As community, we expect more participation.</div><div>As you know its practically not possible to test all scenarios/usecases/setups.</div><div>That is why more regression tests will make sense here.</div><div> </div><div>Thanks<br>Saju</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
<br>
Community Meeting Calendar:<br>
<br>
Schedule -<br>
Every 2nd and 4th Tuesday at 14:30 IST / 09:00 UTC<br>
Bridge: <a href="https://meet.google.com/cpu-eiue-hvk" rel="noreferrer" target="_blank">https://meet.google.com/cpu-eiue-hvk</a><br>
Gluster-users mailing list<br>
<a href="mailto:Gluster-users@gluster.org" target="_blank">Gluster-users@gluster.org</a><br>
<a href="https://lists.gluster.org/mailman/listinfo/gluster-users" rel="noreferrer" target="_blank">https://lists.gluster.org/mailman/listinfo/gluster-users</a><br>
<br>
</blockquote></div></div>