<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: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"><span class="">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: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"><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: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"><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:0 0 0 .8ex;border-left:1px #ccc solid;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:0 0 0 .8ex;border-left:1px #ccc solid;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:0 0 0 .8ex;border-left:1px #ccc solid;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:0 0 0 .8ex;border-left:1px #ccc solid;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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"></div><div class="gmail_extra"><div><div class="m_-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:0 0 0 .8ex;border-left:1px #ccc solid;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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><div><div class="m_-7542020224618914637m_-37148976145158846m_-2343808649619158118h5"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;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_-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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_-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_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722HOEnZb"><font color="#888888"><br></font></span></div><span class="m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722HOEnZb"><font color="#888888"><div><br>-- <br><div class="m_-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_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169m_-87364812102004722gmail_signature" data-smartmail="gmail_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_-7542020224618914637m_-37148976145158846m_-2343808649619158118HOEnZb"><font color="#888888">-- <br><div class="m_-7542020224618914637m_-37148976145158846m_-2343808649619158118m_-1077958629967584169gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Pranith<br></div></div>
</font></span></div>
</blockquote></span></div><span class="m_-7542020224618914637m_-37148976145158846HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-7542020224618914637m_-37148976145158846m_-2343808649619158118gmail_signature" data-smartmail="gmail_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_-7542020224618914637HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-7542020224618914637m_-37148976145158846gmail_signature" data-smartmail="gmail_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="HOEnZb"><font color="#888888"><br></font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_-7542020224618914637gmail_signature" data-smartmail="gmail_signature">Raghavendra G<br></div>
</font></span></div></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Pranith<br></div></div>
</div></div>