<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 26, 2017 at 7:45 PM, Ashish Pandey <span dir="ltr"><<a href="mailto:aspandey@redhat.com" target="_blank">aspandey@redhat.com</a>></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>Xavi,<br></div><div><br></div><div>shd has been disabled in this test on line number 12 and we have also disabled client side heal.<br></div><div>So, no body is going to try to heal it.<br></div></div></div></blockquote><div><br></div><div>Already enqueued heals should be healed. I am taking a look at it. Let's see.<br></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>Ashish<br></div><div><br></div><hr id="m_9167528640027069204zwchr"><div style="color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><b>From: </b>"Atin Mukherjee" <<a href="mailto:amukherj@redhat.com" target="_blank">amukherj@redhat.com</a>><br><b>To: </b>"Ashish Pandey" <<a href="mailto:aspandey@redhat.com" target="_blank">aspandey@redhat.com</a>>, "Raghavendra Gowdappa" <<a href="mailto:rgowdapp@redhat.com" target="_blank">rgowdapp@redhat.com</a>>, "Xavier Hernandez" <<a href="mailto:xhernandez@datalab.es" target="_blank">xhernandez@datalab.es</a>><br><b>Cc: </b>"Gluster Devel" <<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a>><br><b>Sent: </b>Thursday, January 26, 2017 5:50:00 PM<br><b>Subject: </b>Re: [Gluster-devel] Spurious regression failure? tests/basic/ec/ec-background-<wbr>heals.t<div><div class="h5"><br><div><br></div><div>I've +1ed it now.</div><div><br><div class="gmail_quote"><div>On Thu, 26 Jan 2017 at 15:05, Xavier Hernandez <<a href="mailto:xhernandez@datalab.es" target="_blank">xhernandez@datalab.es</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Atin,<br class="m_9167528640027069204gmail_msg"> <br class="m_9167528640027069204gmail_msg"> I don't clearly see what's the problem. Even if the truncate causes a<br class="m_9167528640027069204gmail_msg"> dirty flag to be set, eventually it should be removed before the<br class="m_9167528640027069204gmail_msg"> $HEAL_TIMEOUT value.<br class="m_9167528640027069204gmail_msg"> <br class="m_9167528640027069204gmail_msg"> For now I've marked the test as bad.<br class="m_9167528640027069204gmail_msg"> <br class="m_9167528640027069204gmail_msg"> Patch is: <a href="https://review.gluster.org/16470" rel="noreferrer" class="m_9167528640027069204gmail_msg" target="_blank">https://review.gluster.org/<wbr>16470</a><br class="m_9167528640027069204gmail_msg"> <br class="m_9167528640027069204gmail_msg"> Xavi<br class="m_9167528640027069204gmail_msg"> <br class="m_9167528640027069204gmail_msg"> On 25/01/17 17:24, Atin Mukherjee wrote:<br class="m_9167528640027069204gmail_msg"> > Can we please address this as early as possible, my patch has hit this<br class="m_9167528640027069204gmail_msg"> > failure 3 out of 4 recheck attempts now. I'm guessing some recent<br class="m_9167528640027069204gmail_msg"> > changes has caused it.<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > On Wed, 25 Jan 2017 at 12:10, Ashish Pandey <<a href="mailto:aspandey@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">aspandey@redhat.com</a><br class="m_9167528640027069204gmail_msg"> > <mailto:<a href="mailto:aspandey@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">aspandey@redhat.com</a>>> wrote:<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > Pranith,<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > In this test tests/basic/ec/ec-background-<wbr>heals.t, I think the line<br class="m_9167528640027069204gmail_msg"> > number 86 actually creating a heal entry instead of<br class="m_9167528640027069204gmail_msg"> > helping data heal quickly. What if all the data was already healed<br class="m_9167528640027069204gmail_msg"> > at that moment, truncate came and in preop set the dirty flag and at the<br class="m_9167528640027069204gmail_msg"> > end, as part of the heal, dirty flag was unset on previous good<br class="m_9167528640027069204gmail_msg"> > bricks only and the brick which acted as heal-sink still has dirty<br class="m_9167528640027069204gmail_msg"> > marked by truncate.<br class="m_9167528640027069204gmail_msg"> > That is why we are only seeing "1" as get_pending_heal_count. If a<br class="m_9167528640027069204gmail_msg"> > file was actually not healed it should be "2".<br class="m_9167528640027069204gmail_msg"> > If heal on this file completes and unset of dirty flag happens<br class="m_9167528640027069204gmail_msg"> > before truncate everything will be fine.<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > I think we can wait for file to be heal without truncate?<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > 71 #Test that disabling background-heals still drains the queue<br class="m_9167528640027069204gmail_msg"> > 72 TEST $CLI volume set $V0 disperse.background-heals 1<br class="m_9167528640027069204gmail_msg"> > 73 TEST touch $M0/{a,b,c,d}<br class="m_9167528640027069204gmail_msg"> > 74 TEST kill_brick $V0 $H0 $B0/${V0}2<br class="m_9167528640027069204gmail_msg"> > 75 EXPECT_WITHIN $CONFIG_UPDATE_TIMEOUT "1" mount_get_option_value<br class="m_9167528640027069204gmail_msg"> > $M0 $V0-disperse-0 background-heals<br class="m_9167528640027069204gmail_msg"> > 76 EXPECT_WITHIN $CONFIG_UPDATE_TIMEOUT "200"<br class="m_9167528640027069204gmail_msg"> > mount_get_option_value $M0 $V0-disperse-0 heal-wait-qlength<br class="m_9167528640027069204gmail_msg"> > 77 TEST truncate -s 1GB $M0/a<br class="m_9167528640027069204gmail_msg"> > 78 echo abc > $M0/b<br class="m_9167528640027069204gmail_msg"> > 79 echo abc > $M0/c<br class="m_9167528640027069204gmail_msg"> > 80 echo abc > $M0/d<br class="m_9167528640027069204gmail_msg"> > 81 TEST $CLI volume start $V0 force<br class="m_9167528640027069204gmail_msg"> > 82 EXPECT_WITHIN $CHILD_UP_TIMEOUT "3" ec_child_up_count $V0 0<br class="m_9167528640027069204gmail_msg"> > 83 TEST chown root:root $M0/{a,b,c,d}<br class="m_9167528640027069204gmail_msg"> > 84 TEST $CLI volume set $V0 disperse.background-heals 0<br class="m_9167528640027069204gmail_msg"> > 85 EXPECT_NOT "0" mount_get_option_value $M0 $V0-disperse-0<br class="m_9167528640027069204gmail_msg"> > heal-waiters<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > 86 TEST truncate -s 0 $M0/a # This completes the heal fast ;-) <<<<<<<<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > 87 EXPECT_WITHIN $HEAL_TIMEOUT "^0$" get_pending_heal_count $V0<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > ----<br class="m_9167528640027069204gmail_msg"> > Ashish<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > -----------------------------<wbr>------------------------------<wbr>-------------<br class="m_9167528640027069204gmail_msg"> > *From: *"Raghavendra Gowdappa" <<a href="mailto:rgowdapp@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">rgowdapp@redhat.com</a><br class="m_9167528640027069204gmail_msg"> > <mailto:<a href="mailto:rgowdapp@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">rgowdapp@redhat.com</a>>><br class="m_9167528640027069204gmail_msg"> > *To: *"Nithya Balachandran" <<a href="mailto:nbalacha@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">nbalacha@redhat.com</a><br class="m_9167528640027069204gmail_msg"> > <mailto:<a href="mailto:nbalacha@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">nbalacha@redhat.com</a>>><br class="m_9167528640027069204gmail_msg"> > *Cc: *"Gluster Devel" <<a href="mailto:gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">gluster-devel@gluster.org</a><br class="m_9167528640027069204gmail_msg"> > <mailto:<a href="mailto:gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">gluster-devel@<wbr>gluster.org</a>>>, "Pranith Kumar Karampuri"<br class="m_9167528640027069204gmail_msg"> > <<a href="mailto:pkarampu@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">pkarampu@redhat.com</a> <mailto:<a href="mailto:pkarampu@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">pkarampu@redhat.com</a>>>, "Ashish Pandey"<br class="m_9167528640027069204gmail_msg"> > <<a href="mailto:aspandey@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">aspandey@redhat.com</a> <mailto:<a href="mailto:aspandey@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">aspandey@redhat.com</a>>><br class="m_9167528640027069204gmail_msg"> > *Sent: *Wednesday, January 25, 2017 9:41:38 AM<br class="m_9167528640027069204gmail_msg"> > *Subject: *Re: [Gluster-devel] Spurious regression<br class="m_9167528640027069204gmail_msg"> > failure? tests/basic/ec/ec-background-<wbr>heals.t<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > Found another failure on same test:<br class="m_9167528640027069204gmail_msg"> > <a href="https://build.gluster.org/job/centos6-regression/2874/consoleFull" rel="noreferrer" class="m_9167528640027069204gmail_msg" target="_blank">https://build.gluster.org/<wbr>job/centos6-regression/2874/<wbr>consoleFull</a><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > ----- Original Message -----<br class="m_9167528640027069204gmail_msg"> > > From: "Nithya Balachandran" <<a href="mailto:nbalacha@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">nbalacha@redhat.com</a><br class="m_9167528640027069204gmail_msg"> > <mailto:<a href="mailto:nbalacha@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">nbalacha@redhat.com</a>>><br class="m_9167528640027069204gmail_msg"> > > To: "Gluster Devel" <<a href="mailto:gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">gluster-devel@gluster.org</a><br class="m_9167528640027069204gmail_msg"> > <mailto:<a href="mailto:gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">gluster-devel@<wbr>gluster.org</a>>>, "Pranith Kumar Karampuri"<br class="m_9167528640027069204gmail_msg"> > <<a href="mailto:pkarampu@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">pkarampu@redhat.com</a> <mailto:<a href="mailto:pkarampu@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">pkarampu@redhat.com</a>>>, "Ashish Pandey"<br class="m_9167528640027069204gmail_msg"> > > <<a href="mailto:aspandey@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">aspandey@redhat.com</a> <mailto:<a href="mailto:aspandey@redhat.com" class="m_9167528640027069204gmail_msg" target="_blank">aspandey@redhat.com</a>>><br class="m_9167528640027069204gmail_msg"> > > Sent: Tuesday, January 24, 2017 9:16:31 AM<br class="m_9167528640027069204gmail_msg"> > > Subject: [Gluster-devel] Spurious regression<br class="m_9167528640027069204gmail_msg"> > failure? tests/basic/ec/ec-background-<wbr>heals.t<br class="m_9167528640027069204gmail_msg"> > ><br class="m_9167528640027069204gmail_msg"> > > Hi,<br class="m_9167528640027069204gmail_msg"> > ><br class="m_9167528640027069204gmail_msg"> > ><br class="m_9167528640027069204gmail_msg"> > > Can you please take a look at<br class="m_9167528640027069204gmail_msg"> > > <a href="https://build.gluster.org/job/centos6-regression/2859/console" rel="noreferrer" class="m_9167528640027069204gmail_msg" target="_blank">https://build.gluster.org/job/<wbr>centos6-regression/2859/<wbr>console</a> ?<br class="m_9167528640027069204gmail_msg"> > ><br class="m_9167528640027069204gmail_msg"> > > tests/basic/ec/ec-background-<wbr>heals.t has failed.<br class="m_9167528640027069204gmail_msg"> > ><br class="m_9167528640027069204gmail_msg"> > > Thanks,<br class="m_9167528640027069204gmail_msg"> > > Nithya<br class="m_9167528640027069204gmail_msg"> > ><br class="m_9167528640027069204gmail_msg"> > > ______________________________<wbr>_________________<br class="m_9167528640027069204gmail_msg"> > > Gluster-devel mailing list<br class="m_9167528640027069204gmail_msg"> > > <a href="mailto:Gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">Gluster-devel@gluster.org</a> <mailto:<a href="mailto:Gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">Gluster-devel@gluster.<wbr>org</a>><br class="m_9167528640027069204gmail_msg"> > > <a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" class="m_9167528640027069204gmail_msg" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/gluster-devel</a><br class="m_9167528640027069204gmail_msg"> > _____________________________<wbr>__________________<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > Gluster-devel mailing list<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > <a href="mailto:Gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">Gluster-devel@gluster.org</a> <mailto:<a href="mailto:Gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">Gluster-devel@gluster.<wbr>org</a>><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > <a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" class="m_9167528640027069204gmail_msg" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/gluster-devel</a><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > --<br class="m_9167528640027069204gmail_msg"> > - Atin (atinm)<br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> > ______________________________<wbr>_________________<br class="m_9167528640027069204gmail_msg"> > Gluster-devel mailing list<br class="m_9167528640027069204gmail_msg"> > <a href="mailto:Gluster-devel@gluster.org" class="m_9167528640027069204gmail_msg" target="_blank">Gluster-devel@gluster.org</a><br class="m_9167528640027069204gmail_msg"> > <a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" class="m_9167528640027069204gmail_msg" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/gluster-devel</a><br class="m_9167528640027069204gmail_msg"> ><br class="m_9167528640027069204gmail_msg"> <br class="m_9167528640027069204gmail_msg"></blockquote></div></div><div dir="ltr">-- <br></div><div>- Atin (atinm)</div></div></div></div><div><br></div></div></div><br>______________________________<wbr>_________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org">Gluster-devel@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/gluster-devel</a><br></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>