[Gluster-devel] Snapshot test Spurious Failure

Joseph Fernandes josferna at redhat.com
Fri Sep 4 13:31:05 UTC 2015

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.


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 

More information about the Gluster-devel mailing list