[Gluster-devel] Snapshot test Spurious Failure

[Joseph Fernandes]

Thanks Avra! will do the appropriate change.

----- Original Message -----
From: "Avra Sengupta" <asengupt at redhat.com>
To: "Joseph Fernandes" <josferna at redhat.com>
Cc: "Gluster Devel" <gluster-devel at gluster.org>
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 
>>
>>
>