[Gluster-devel] ./tests/bitrot/bug-1373520.t failure on master

[Pranith Kumar Karampuri]

Thanks for the work Atin/Krutika. My analysis is wrong. I need to also see
why lookup on one file lead to lookup on its hardlink. Which I will do
later.

On Wed, Mar 1, 2017 at 12:41 PM, Krutika Dhananjay <kdhananj at redhat.com>
wrote:

> Found the bug. Please see https://review.gluster.org/#/
> c/16462/5/xlators/storage/posix/src/posix-handle.c at 977
>
> Will be posting the fix in some time.
>
> -Krutika
>
> On Tue, Feb 28, 2017 at 5:45 PM, Atin Mukherjee <amukherj at redhat.com>
> wrote:
>
>>
>>
>> On Tue, Feb 28, 2017 at 5:10 PM, Atin Mukherjee <amukherj at redhat.com>
>> wrote:
>>
>>> Can this patch be reverted asap as it has blocked other patches to get
>>> in? IMO, we shouldn't be marking the test as bad given we know the patch
>>> which introduced the regression.
>>>
>>
>> I've posted https://review.gluster.org/#/c/16787 to revert the change.
>> Can this be merged once it passes regression?
>>
>>
>>>
>>> On Tue, Feb 28, 2017 at 4:16 PM, Atin Mukherjee <amukherj at redhat.com>
>>> wrote:
>>>
>>>> https://review.gluster.org/16462 has caused this regression as per the
>>>> git bisect.
>>>>
>>>> On Tue, Feb 28, 2017 at 3:28 PM, Pranith Kumar Karampuri <
>>>> pkarampu at redhat.com> wrote:
>>>>
>>>>> This is being tracked @ https://bugzilla.redhat.com/sh
>>>>> ow_bug.cgi?id=1427404, krutika posted a patch to move it to bad tests
>>>>> until we find why a lookup on one file is leading to lookup on the hardlink
>>>>> instead.
>>>>>
>>>>> On Tue, Feb 28, 2017 at 2:56 PM, Susant Palai <spalai at redhat.com>
>>>>> wrote:
>>>>>
>>>>>> Hi,
>>>>>>    test case: ./tests/bitrot/bug-1373520.t is seen to be failing on
>>>>>> different regression runs on master.
>>>>>> Requesting to look in to it.
>>>>>>
>>>>>> Few instances:
>>>>>> https://build.gluster.org/job/netbsd7-regression/3118/consoleFull
>>>>>> https://build.gluster.org/job/centos6-regression/3451/
>>>>>> https://build.gluster.org/job/netbsd7-regression/3122/consoleFull
>>>>>>
>>>>>> Thanks,
>>>>>> Susant
>>>>>> _______________________________________________
>>>>>> Gluster-devel mailing list
>>>>>> Gluster-devel at gluster.org
>>>>>> http://lists.gluster.org/mailman/listinfo/gluster-devel
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Pranith
>>>>>
>>>>> _______________________________________________
>>>>> Gluster-devel mailing list
>>>>> Gluster-devel at gluster.org
>>>>> http://lists.gluster.org/mailman/listinfo/gluster-devel
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>>
>>>> ~ Atin (atinm)
>>>>
>>>
>>>
>>>
>>> --
>>>
>>> ~ Atin (atinm)
>>>
>>
>>
>>
>> --
>>
>> ~ Atin (atinm)
>>
>> _______________________________________________
>> Gluster-devel mailing list
>> Gluster-devel at gluster.org
>> http://lists.gluster.org/mailman/listinfo/gluster-devel
>>
>
>


-- 
Pranith
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gluster.org/pipermail/gluster-devel/attachments/20170301/df05431a/attachment-0001.html>