<p dir="ltr">Raghavendra,</p>
<p dir="ltr">No problem. As you said , glusterd_quota_limit_usage invokes the function which regenerates the conf file. Though I do not remember exactly, to my understanding when I tried, it did not work properly in my setup. It is apparently because in the later function where we regenerate the quota.conf for versions greater than or equal to 3.7, when it is setting a limit or ideally when you are resetting a limit, it searches for the gfid on which it needs to set/reset the limit and modify only that to 17 bytes leaving the remaining ones untouched which again would result in unexpected behavior. In the case of enable or disable, the entire file gets newly generated. With this patch, we have done that during an upgrade as well. </p>
<p dir="ltr">Even I am not completely sure. Anyways its better to test and confirm the fact. I can test the same over the weekend if it&#39;s fine.</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Nov 10, 2016 9:00 PM, &quot;Raghavendra G&quot; &lt;<a href="mailto:raghavendra@gluster.com">raghavendra@gluster.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 10, 2016 at 8:46 PM, Manikandan Selvaganesh <span dir="ltr">&lt;<a href="mailto:manikandancs333@gmail.com" target="_blank">manikandancs333@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><p dir="ltr">Enabling/disabling quota or removing limits are the ways in which quota.conf is regenerated to the later version. It works properly. And as Pranith said, both enabling/disabling takes a lot of time to crawl(though now much faster with enhanced quota enable/disable process) which we cannot suggest the users with a lot of quota configuration. Resetting the limit using limit-usage does not work properly. I have tested the same. The workaround is based on the user setup here. I mean the steps he exactly used in order matters here. The workaround is not so generic.</p></blockquote><div><br></div><div>Thanks Manikandan for the reply :). I&#39;ve not tested this, but went through the code. If I am not wrong, function glusterd_store_quota_config  would write a quota.conf which is compatible for versions &gt;= 3.7. This function is invoked by glusterd_quota_limit_usage unconditionally in success path. What am I missing here?<br><br></div><div>@Pranith,<br><br></div><div>Since Manikandan says his tests didn&#39;t succeed always, probably we should do any of the following<br></div><div>1. hold back the release till we successfully test limit-usage to rewrite quota.conf (I can do this tomorrow)<br></div><div>2. get the patch in question for 3.9<br></div><div>3. If 1 is failing, debug why 1 is not working and fix that.<br></div><div><br></div><div>regards,<br></div><div>Raghavendra<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><p dir="ltr"> However, quota enable/disable would regenerate the file on any case. </p>
<p dir="ltr">IMO, this bug is critical. I am not sure though how often users would hit this - Updating from 3.6 to latest versions. From 3.7 to latest, its fine, this has nothing to do with this patch.</p><div class="m_-3584478918523020132gmail-HOEnZb"><div class="m_-3584478918523020132gmail-h5">
<div class="gmail_extra"><br><div class="gmail_quote">On Nov 10, 2016 8:03 PM, &quot;Pranith Kumar Karampuri&quot; &lt;<a href="mailto:pkarampu@redhat.com" target="_blank">pkarampu@redhat.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 10, 2016 at 7:43 PM, Raghavendra G <span dir="ltr">&lt;<a href="mailto:raghavendra@gluster.com" target="_blank">raghavendra@gluster.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Thu, Nov 10, 2016 at 2:14 PM, Pranith Kumar Karampuri <span dir="ltr">&lt;<a href="mailto:pkarampu@redhat.com" target="_blank">pkarampu@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Thu, Nov 10, 2016 at 1:11 PM, Atin Mukherjee <span dir="ltr">&lt;<a href="mailto:amukherj@redhat.com" target="_blank">amukherj@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Thu, Nov 10, 2016 at 1:04 PM, Pranith Kumar Karampuri <span dir="ltr">&lt;<a href="mailto:pkarampu@redhat.com" target="_blank">pkarampu@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I am trying to understand the criticality of these patches. Raghavendra&#39;s patch is crucial because gfapi workloads(for samba and qemu) are affected severely. I waited for Krutika&#39;s patch because VM usecase can lead to disk corruption on replace-brick. If you could let us know the criticality and we are in agreement that they are this severe, we can definitely take them in. Otherwise next release is better IMO. Thoughts?<br></div></blockquote><div><br></div></span><div>If you are asking about how critical they are, then the first two are definitely not but third one is actually a critical one as if user upgrades from 3.6 to latest with quota enable, further peer probes get rejected and the only work around is to disable quota and re-enable it back.<br></div></div></div></div></blockquote><div><br></div></span><div>Let me take Raghavendra G&#39;s input also here.<br><br></div><div>Raghavendra, what do you think we should do? Merge it or live with it till 3.9.1?<br></div></div></div></div></blockquote><div><br></div></span><div>The commit says quota.conf is rewritten to compatible version during three operations:<br></div><div>1. enable/disable quota<br></div></div></div></div></blockquote><div><br>This will involve crawling the whole FS doesn&#39;t it? <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div>2. limit usage<br></div></div></div></div></blockquote><div><br></div><div>This is a good way IMO. Could Sanoj/you confirm that this works once by testing it.<br></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"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div>3. remove quota limit<br></div></div></div></div></blockquote><div><br></div><div>I guess you added this for completeness. We can&#39;t really suggest this to users as a work around.<br></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"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>I checked the code and it works as stated in commit msg. Probably we can list the above three operations as work around and take this patch in for 3.9.1<br></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><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"><div><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678h5"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>On a different note, 3.9 head is not static and moving forward. So if you are really looking at only critical patches need to go in, that&#39;s not happening, just a word of caution!<br><br></div><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"></div><div class="gmail_extra"><div><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118h5"><br><div class="gmail_quote">On Thu, Nov 10, 2016 at 12:56 PM, Atin Mukherjee <span dir="ltr">&lt;<a href="mailto:amukherj@redhat.com" target="_blank">amukherj@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div>Pranith,<br><br></div>I&#39;d like to see following patches getting in:<br><br><a href="http://review.gluster.org/#/c/15722/" target="_blank">http://review.gluster.org/#/c/<wbr>15722/</a><br><a href="http://review.gluster.org/#/c/15714/" target="_blank">http://review.gluster.org/#/c/<wbr>15714/</a><br><a href="http://review.gluster.org/#/c/15792/" target="_blank">http://review.gluster.org/#/c/<wbr>15792/</a> <br></div></div></blockquote></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail_extra"><div><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118h5"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br><br></div><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169h5">On Thu, Nov 10, 2016 at 7:12 AM, Pranith Kumar Karampuri <span dir="ltr">&lt;<a href="mailto:pkarampu@redhat.com" target="_blank">pkarampu@redhat.com</a>&gt;</span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169h5"><div dir="ltr"><div><div><div>hi,<br></div>      The only problem left was EC taking more time. This should affect small files a lot more. Best way to solve it is using compound-fops. So for now I think going ahead with the release is best.<br><br></div></div>We are waiting for Raghavendra Talur&#39;s <a href="http://review.gluster.org/#/c/15778" target="_blank">http://review.gluster.org/#/c/<wbr>15778</a> before going ahead with the release. If we missed any other crucial patch please let us know.<br clear="all"><div><div><div><div><br></div><div>Will make the release as soon as this patch is merged.<span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722HOEnZb"><font color="#888888"><br></font></span></div><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722HOEnZb"><font color="#888888"><div><br>-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722m_-4684222911575648709gmail_signature"><div dir="ltr">Pranith &amp; Aravinda<br></div></div>
</div></font></span></div></div></div></div>
<br></div></div>______________________________<wbr>_________________<br>
maintainers mailing list<br>
<a href="mailto:maintainers@gluster.org" target="_blank">maintainers@gluster.org</a><br>
<a href="http://www.gluster.org/mailman/listinfo/maintainers" rel="noreferrer" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/maintainers</a><br>
<br></blockquote></div><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722gmail_signature"><div dir="ltr"><div><div dir="ltr"><br></div><div>~ Atin (atinm)<br></div></div></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><br></div></div><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118HOEnZb"><font color="#888888">-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169gmail_signature"><div dir="ltr">Pranith<br></div></div>
</font></span></div>
</blockquote></span></div><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846m_-2343808649619158118gmail_signature"><div dir="ltr"><div><div dir="ltr"><br></div><div>~ Atin (atinm)<br></div></div></div></div>
</font></span></div></div>
</blockquote></span></div><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637m_-37148976145158846gmail_signature"><div dir="ltr">Pranith<br></div></div>
</font></span></div></div>
<br></div></div>______________________________<wbr>_________________<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" rel="noreferrer" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/gluster-devel</a><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678HOEnZb"><font color="#888888"><br></font></span></blockquote></div><span class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678m_-7542020224618914637gmail_signature">Raghavendra G<br></div>
</font></span></div></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="m_-3584478918523020132gmail-m_8102323529812506398m_6777851230008887678gmail_signature"><div dir="ltr">Pranith<br></div></div>
</div></div>
<br>______________________________<wbr>_________________<br>
maintainers mailing list<br>
<a href="mailto:maintainers@gluster.org" target="_blank">maintainers@gluster.org</a><br>
<a href="http://www.gluster.org/mailman/listinfo/maintainers" rel="noreferrer" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/maintainers</a><br>
<br></blockquote></div></div>
</div></div><br>______________________________<wbr>_________________<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" rel="noreferrer" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/gluster-devel</a><br></blockquote></div><br><br clear="all"><br>-- <br><div class="m_-3584478918523020132gmail_signature">Raghavendra G<br></div>
</div></div>
</blockquote></div></div>