[Gluster-devel] /tests/bugs/quota/bug-1153964.t is consistently failing

Wed Jun 24 06:12:19 UTC 2015

http://review.gluster.org/#/c/11362/ has been merged.

----- Original Message -----
> From: "Atin Mukherjee" <amukherj at redhat.com>
> To: "Raghavendra Gowdappa" <rgowdapp at redhat.com>
> Cc: "Niels de Vos" <ndevos at redhat.com>, "Vijaikumar M" <vmallika at redhat.com>, "Raghavendra G"
> <raghavendra at gluster.com>, "Gluster Devel" <gluster-devel at gluster.org>
> Sent: Wednesday, June 24, 2015 10:55:02 AM
> Subject: Re: [Gluster-devel] /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 at redhat.com>
> >> To: "Niels de Vos" <ndevos at redhat.com>, "Vijaikumar M"
> >> <vmallika at redhat.com>
> >> Cc: "Raghavendra G" <raghavendra at gluster.com>, "Gluster Devel"
> >> <gluster-devel at gluster.org>
> >> Sent: Wednesday, June 24, 2015 10:15:12 AM
> >> Subject: Re: [Gluster-devel] /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 at gmail.com
> >>>>>>> <mailto:yknev.shankar at gmail.com>> wrote:
> >>>>>>>
> >>>>>>>    OK. Two reverts of the same patch ;)
> >>>>>>>
> >>>>>>>    Pick one.
> >>>>>>>
> >>>>>>>    On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa
> >>>>>>>    <rgowdapp at redhat.com <mailto:rgowdapp at redhat.com>> 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 at datalab.es
> >>>>>>>    <mailto:xhernandez at datalab.es>>
> >>>>>>>    >> To: "Gluster Devel" <gluster-devel at gluster.org
> >>>>>>>    <mailto:gluster-devel at gluster.org>>
> >>>>>>>    >> Sent: Tuesday, June 23, 2015 12:44:47 PM
> >>>>>>>    >> Subject: [Gluster-devel] /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 at gluster.org <mailto:Gluster-devel at gluster.org>
> >>>>>>>    >> http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>>>>    >>
> >>>>>>>    > _______________________________________________
> >>>>>>>    > Gluster-devel mailing list
> >>>>>>>    > Gluster-devel at gluster.org <mailto:Gluster-devel at gluster.org>
> >>>>>>>    > http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>>>>    _______________________________________________
> >>>>>>>    Gluster-devel mailing list
> >>>>>>>    Gluster-devel at gluster.org <mailto:Gluster-devel at gluster.org>
> >>>>>>>    http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> --
> >>>>>>> Raghavendra G
> >>>>>>>
> >>>>>>>
> >>>>>>> _______________________________________________
> >>>>>>> Gluster-devel mailing list
> >>>>>>> Gluster-devel at gluster.org
> >>>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>>> _______________________________________________
> >>>>>> Gluster-devel mailing list
> >>>>>> Gluster-devel at gluster.org
> >>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>>
> >>> _______________________________________________
> >>> Gluster-devel mailing list
> >>> Gluster-devel at gluster.org
> >>> http://www.gluster.org/mailman/listinfo/gluster-devel
> >>>
> >>
> >> --
> >> ~Atin
> >> _______________________________________________
> >> Gluster-devel mailing list
> >> Gluster-devel at gluster.org
> >> http://www.gluster.org/mailman/listinfo/gluster-devel
> >>
> 
> --
> ~Atin
> 