Re: [Gluster-devel] 3.3 requires extended attribute on /

[Krishnan Parthasarathi]

Emmanuel, 

I have submitted the fix for review: http://review.gluster.com/3380
I have not tested the fix with "/" having EA disabled. It would be great
if you could confirm the looping forever doesn't happen with this fix.

thanks,
krish

----- Original Message -----
From: "Emmanuel Dreyfus" <address@hidden>
To: "Krishnan Parthasarathi" <address@hidden>
Cc: address@hidden
Sent: Sunday, May 20, 2012 1:41:04 PM
Subject: Re: [Gluster-devel] 3.3 requires extended attribute on /

Krishnan Parthasarathi <address@hidden> wrote:

> It is strange that the you see glusterd_path_in_use() loop forever. If I
> am not wrong, the inner loop checks for presence of trusted.gfid and
> trusted.glusterfs.volume-id and should exit after that, and the outer loop
> performs dirname on the path repeatedly and dirname(3) guarantees such an
> operation should return "/" eventually, which we check.

Here is my setup when I tried that:
/  with EA enabled
/export/wd3a ibrick with EA enabled

But I may have been testing with an untintended patch in
glusterd_path_in_use(). I will retry with the right fix once it will be
available.


-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
address@hidden