----- Original Message ----- > From: "Sachin Pandit" <spandit@xxxxxxxxxx> > To: "Xavier Hernandez" <xhernandez@xxxxxxxxxx> > Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>, "Raghavendra G" <raghavendra@xxxxxxxxxxx> > Sent: Thursday, June 25, 2015 5:50:01 PM > Subject: Re: /tests/bugs/quota/bug-1153964.t is consistently failing > > Hi Xavier, > > I did a git pull and I could see the commit > "01e42f25ebdc44847e8b1dce02f7fd486b40dbc2" > in my repository. This commit is the revert of the commit > "8ab6608accb62d6320d1fc1fbe651fcafd376270" > > ~ Sachin. > > ----- Original Message ----- > > From: "Xavier Hernandez" <xhernandez@xxxxxxxxxx> > > To: "Vijaikumar M" <vmallika@xxxxxxxxxx>, "Gluster Devel" > > <gluster-devel@xxxxxxxxxxx> > > Cc: "Raghavendra G" <raghavendra@xxxxxxxxxxx> > > Sent: Thursday, June 25, 2015 5:41:14 PM > > Subject: Re: /tests/bugs/quota/bug-1153964.t is > > consistently failing > > > > It seems that the patch has not been correctly committed into master. I > > see it as merged in gerrit, but doing a 'git pull' on master on a local > > repository doesn't seem to include it. And it looks like [1] was rebased on Jun 24 at "10:02 AM" and the reverted patch [2] got merged on Jun 24 at "11:41 AM". That is the reason why [1] is failing the same test case. [1] http://review.gluster.org/#/c/10465/ [2] http://review.gluster.org/#/c/11362/ ~ Sachin. > > > > Am I doing something wrong ? > > > > It also seems that a recently rebased patch is still failing because of > > the same problem: > > > > http://build.gluster.org/job/rackspace-regression-2GB-triggered/11258/consoleFull > > > > Xavi > > > > On 06/24/2015 08:21 AM, Vijaikumar M wrote: > > > Hi All, > > > > > > I request you to re-base your patch which are failed regression with > > > test-case bug-1153964.t > > > > > > Thanks, > > > Vijay > > > > > > > > > On Wednesday 24 June 2015 11:42 AM, Raghavendra Gowdappa wrote: > > >> http://review.gluster.org/#/c/11362/ has been merged. > > >> > > >> ----- Original Message ----- > > >>> From: "Atin Mukherjee" <amukherj@xxxxxxxxxx> > > >>> To: "Raghavendra Gowdappa" <rgowdapp@xxxxxxxxxx> > > >>> Cc: "Niels de Vos" <ndevos@xxxxxxxxxx>, "Vijaikumar M" > > >>> <vmallika@xxxxxxxxxx>, "Raghavendra G" > > >>> <raghavendra@xxxxxxxxxxx>, "Gluster Devel" <gluster-devel@xxxxxxxxxxx> > > >>> Sent: Wednesday, June 24, 2015 10:55:02 AM > > >>> Subject: Re: /tests/bugs/quota/bug-1153964.t is > > >>> consistently failing > > >>> > > >>> > > >>> > > >>> On 06/24/2015 10:53 AM, Raghavendra Gowdappa wrote: > > >>>> > > >>>> ----- Original Message ----- > > >>>>> From: "Atin Mukherjee" <amukherj@xxxxxxxxxx> > > >>>>> To: "Niels de Vos" <ndevos@xxxxxxxxxx>, "Vijaikumar M" > > >>>>> <vmallika@xxxxxxxxxx> > > >>>>> Cc: "Raghavendra G" <raghavendra@xxxxxxxxxxx>, "Gluster Devel" > > >>>>> <gluster-devel@xxxxxxxxxxx> > > >>>>> Sent: Wednesday, June 24, 2015 10:15:12 AM > > >>>>> Subject: Re: /tests/bugs/quota/bug-1153964.t is > > >>>>> consistently failing > > >>>>> > > >>>>> When is this patch getting merged, this is blocking other patches > > >>>>> to get > > >>>>> in. > > >>>> Revert of http://review.gluster.org/11311, is waiting for regression > > >>>> runs > > >>>> to pass. There are three patches (duplicates of each other). If > > >>>> anyone of > > >>>> them pass both regression runs, I'll merge them. As far as refcounting > > >>>> mechanism go, it'll take some time to review and merge the patch. > > >>> Once the revert patch is merged, we are good to go. Please let us know > > >>> once it's merged as post that all patches need rebase. > > >>> > > >>> ~Atin > > >>>>> ~Atin > > >>>>> > > >>>>> On 06/23/2015 06:26 PM, Niels de Vos wrote: > > >>>>>> On Tue, Jun 23, 2015 at 05:30:39PM +0530, Vijaikumar M wrote: > > >>>>>>> > > >>>>>>> On Tuesday 23 June 2015 04:28 PM, Niels de Vos wrote: > > >>>>>>>> On Tue, Jun 23, 2015 at 03:45:43PM +0530, Vijaikumar M wrote: > > >>>>>>>>> I have submitted below patch which fixes this issue. I am > > >>>>>>>>> handling > > >>>>>>>>> memory > > >>>>>>>>> clean-up with reference countmechanism. > > >>>>>>>>> > > >>>>>>>>> http://review.gluster.org/#/c/11361 > > >>>>>>>> Is there a reason you can not use the (new) refcounting > > >>>>>>>> functions that > > >>>>>>>> were introduceed with http://review.gluster.org/11022 ? > > >>>>>>> I was not aware that ref-counting patch was merged. Sure we will > > >>>>>>> use > > >>>>>>> these > > >>>>>>> function and re-submit my patch. > > >>>>>> Ok, thanks! > > >>>>>> Niels > > >>>>>> > > >>>>>>> Thanks, > > >>>>>>> Vijay > > >>>>>>> > > >>>>>>> > > >>>>>>>> It would be nicer to standardize all refcounting mechanisms on one > > >>>>>>>> implementation. I hope we can replace existing refcounting with > > >>>>>>>> this > > >>>>>>>> one > > >>>>>>>> too. Introducing more refcounting ways is not going to be helpful. > > >>>>>>>> > > >>>>>>>> Thanks, > > >>>>>>>> Niels > > >>>>>>>> > > >>>>>>>>> Thanks, > > >>>>>>>>> Vijay > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>>> On Tuesday 23 June 2015 12:58 PM, Raghavendra G wrote: > > >>>>>>>>>> Multiple replies to same query. Pick one ;). > > >>>>>>>>>> > > >>>>>>>>>> On Tue, Jun 23, 2015 at 12:55 PM, Venky Shankar > > >>>>>>>>>> <yknev.shankar@xxxxxxxxx > > >>>>>>>>>> <mailto:yknev.shankar@xxxxxxxxx>> wrote: > > >>>>>>>>>> > > >>>>>>>>>> OK. Two reverts of the same patch ;) > > >>>>>>>>>> > > >>>>>>>>>> Pick one. > > >>>>>>>>>> > > >>>>>>>>>> On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa > > >>>>>>>>>> <rgowdapp@xxxxxxxxxx <mailto:rgowdapp@xxxxxxxxxx>> wrote: > > >>>>>>>>>> > Seems like its a memory corruption caused by: > > >>>>>>>>>> > http://review.gluster.org/11311 > > >>>>>>>>>> > > > >>>>>>>>>> > I've reverted the patch at: > > >>>>>>>>>> > http://review.gluster.org/11360 > > >>>>>>>>>> > > > >>>>>>>>>> > ----- Original Message ----- > > >>>>>>>>>> >> From: "Xavier Hernandez" <xhernandez@xxxxxxxxxx > > >>>>>>>>>> <mailto:xhernandez@xxxxxxxxxx>> > > >>>>>>>>>> >> To: "Gluster Devel" <gluster-devel@xxxxxxxxxxx > > >>>>>>>>>> <mailto:gluster-devel@xxxxxxxxxxx>> > > >>>>>>>>>> >> Sent: Tuesday, June 23, 2015 12:44:47 PM > > >>>>>>>>>> >> Subject: > > >>>>>>>>>> /tests/bugs/quota/bug-1153964.t is > > >>>>>>>>>> consistently failing > > >>>>>>>>>> >> > > >>>>>>>>>> >> Hi, > > >>>>>>>>>> >> > > >>>>>>>>>> >> the quota test bug-1153964.t is failing consistently for > > >>>>>>>>>> >> a > > >>>>>>>>>> >> totally > > >>>>>>>>>> >> unrelated patch. Is this a known issue ? > > >>>>>>>>>> >> > > >>>>>>>>>> >> > > >>>>>>>>>> > > >>>>>>>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11142/consoleFull > > >>>>>>>>>> > > >>>>>>>>>> >> > > >>>>>>>>>> > > >>>>>>>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11165/consoleFull > > >>>>>>>>>> > > >>>>>>>>>> >> > > >>>>>>>>>> > > >>>>>>>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11172/consoleFull > > >>>>>>>>>> > > >>>>>>>>>> >> > > >>>>>>>>>> > > >>>>>>>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11191/consoleFull > > >>>>>>>>>> > > >>>>>>>>>> >> > > >>>>>>>>>> >> Xavi > > >>>>>>>>>> >> _______________________________________________ > > >>>>>>>>>> >> Gluster-devel mailing list > > >>>>>>>>>> >> Gluster-devel@xxxxxxxxxxx > > >>>>>>>>>> <mailto:Gluster-devel@xxxxxxxxxxx> > > >>>>>>>>>> >> http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>>>>>>> >> > > >>>>>>>>>> > _______________________________________________ > > >>>>>>>>>> > Gluster-devel mailing list > > >>>>>>>>>> > Gluster-devel@xxxxxxxxxxx > > >>>>>>>>>> <mailto:Gluster-devel@xxxxxxxxxxx> > > >>>>>>>>>> > http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>>>>>>> _______________________________________________ > > >>>>>>>>>> Gluster-devel mailing list > > >>>>>>>>>> Gluster-devel@xxxxxxxxxxx <mailto:Gluster-devel@xxxxxxxxxxx> > > >>>>>>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>>>>>>> > > >>>>>>>>>> > > >>>>>>>>>> > > >>>>>>>>>> > > >>>>>>>>>> -- > > >>>>>>>>>> Raghavendra G > > >>>>>>>>>> > > >>>>>>>>>> > > >>>>>>>>>> _______________________________________________ > > >>>>>>>>>> Gluster-devel mailing list > > >>>>>>>>>> Gluster-devel@xxxxxxxxxxx > > >>>>>>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>>>>>> _______________________________________________ > > >>>>>>>>> Gluster-devel mailing list > > >>>>>>>>> Gluster-devel@xxxxxxxxxxx > > >>>>>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>>> _______________________________________________ > > >>>>>> Gluster-devel mailing list > > >>>>>> Gluster-devel@xxxxxxxxxxx > > >>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>>> > > >>>>> -- > > >>>>> ~Atin > > >>>>> _______________________________________________ > > >>>>> Gluster-devel mailing list > > >>>>> Gluster-devel@xxxxxxxxxxx > > >>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > > >>>>> > > >>> -- > > >>> ~Atin > > >>> > > > > > > _______________________________________________ > > > Gluster-devel mailing list > > > Gluster-devel@xxxxxxxxxxx > > > http://www.gluster.org/mailman/listinfo/gluster-devel > > _______________________________________________ > > Gluster-devel mailing list > > Gluster-devel@xxxxxxxxxxx > > http://www.gluster.org/mailman/listinfo/gluster-devel > > > _______________________________________________ > Gluster-devel mailing list > Gluster-devel@xxxxxxxxxxx > http://www.gluster.org/mailman/listinfo/gluster-devel > _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel
