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. > > 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
