Re: Snapshot test Spurious Failure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Avra! will do the appropriate change.

----- Original Message -----
From: "Avra Sengupta" <asengupt@xxxxxxxxxx>
To: "Joseph Fernandes" <josferna@xxxxxxxxxx>
Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>
Sent: Friday, September 4, 2015 5:45:19 PM
Subject: Re: Snapshot test Spurious Failure

It's not a spurious failure. In the patch 
http://review.gluster.org/#/c/12031/3, you are using list_for_each_entry 
in clear_bricklist(), and deleting an item from the list. That is not a 
right practice. Instead you should use  list_for_each_entry_safe.

Regards,
Avra

On 09/04/2015 11:50 AM, Avra Sengupta wrote:
> Hi,
>
> I am having a look at the core. Will update shortly.
>
> Regards,
> Avra
>
> On 09/04/2015 11:46 AM, Joseph Fernandes wrote:
>>
>> ./tests/bugs/snapshot/bug-1227646.t
>> https://build.gluster.org/job/rackspace-regression-2GB-triggered/14021/consoleFull 
>>
>>
>

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel



[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux