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

Atin Mukherjee amukherj at redhat.com
Wed Jun 24 05:25:02 UTC 2015



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


More information about the Gluster-devel mailing list