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

Xavier Hernandez xhernandez at datalab.es
Thu Jun 25 15:16:59 UTC 2015 

Sorry, my bad. I didn't see an error while doing the pull on my local 
repository and I thought it was updated while it wasn't.

Xavi

On 06/25/2015 02:26 PM, Sachin Pandit wrote:
>
>
> ----- Original Message -----
>> From: "Sachin Pandit" <spandit at redhat.com>
>> To: "Xavier Hernandez" <xhernandez at datalab.es>
>> Cc: "Gluster Devel" <gluster-devel at gluster.org>, "Raghavendra G" <raghavendra at gluster.com>
>> Sent: Thursday, June 25, 2015 5:50:01 PM
>> Subject: Re: [Gluster-devel] /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 at datalab.es>
>>> To: "Vijaikumar M" <vmallika at redhat.com>, "Gluster Devel"
>>> <gluster-devel at gluster.org>
>>> Cc: "Raghavendra G" <raghavendra at gluster.com>
>>> Sent: Thursday, June 25, 2015 5:41:14 PM
>>> Subject: Re: [Gluster-devel] /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 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
>>>>>>
>>>>
>>>> _______________________________________________
>>>> 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
>>

More information about the Gluster-devel mailing list