-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 2014-06-23 12:03, Niels de Vos wrote:
> On Tue, Jun 17, 2014 at 11:49:26AM -0400, Shyamsundar Ranganathan wrote:
>> You maybe looking at the problem being fixed here, [1].
>>
>> On a lookup attribute mismatch was not being healed across
>> directories, and this patch attempts to address the same. Currently
>> the version of the patch does not heal the S_ISUID and S_ISGID bits,
>> which is work in progress (but easy enough to incorporate and test
>> based on the patch at [1]).
>>
>> On a separate note, add-brick just adds a brick to the cluster, the
>> lookup is where the heal (or creation of the directory across all sub
>> volumes in DHT xlator) is being done.
>
> I assume that this is not a regression between 3.5.0 and 3.5.1? If that
> is the case, we can pull the fix in 3.5.2 because 3.5.1 really should
> not get delayed much longer.
No, it does not work in 3.5.0 either :-(
The proposed patch does not work as intended, with the following hieararchy
755 0: 0 /mnt/gluster
2777 0:1000 /mnt/gluster/test
2755 1000:1000 /mnt/gluster/test/dir1
2755 1000:1000 /mnt/gluster/test/dir1/dir2
In the (approx 25%) of cases where my test-script does trigger a
self heal on disk2, 10% ends up with (giving access error on client):
0 0: 0 /data/disk2/gluster/test
755 1000:1000 /data/disk2/gluster/test/dir1
755 1000:1000 /data/disk2/gluster/test/dir1/dir2
or
2777 0:1000 /data/disk2/gluster/test
0 0: 0 /data/disk2/gluster/test/dir1
755 1000:1000 /data/disk2/gluster/test/dir1/dir2
or
2777 0:1000 /data/disk2/gluster/test
2755 1000:1000 /data/disk2/gluster/test/dir1
0 0: 0 /data/disk2/gluster/test/dir1/dir2
and 73% ends up with either partially healed directories
(/data/disk2/gluster/test/dir1/dir2 or
/data/disk2/gluster/test/dir1 missing) or the sgid bit
[randomly] set on some of the directories.
Since I don't even understand how to reliably trigger
a self-heal of the directories, I'm currently clueless
to the reason for this behaviour.
Soo, I think that the comment from susant in
http://review.gluster.org/#/c/6983/3/xlators/cluster/dht/src/dht-common.c:
susant palai Jun 13 9:04 AM
I think we dont have to worry about that.
Rebalance does not interfere with directory SUID/GID/STICKY bits.
unfortunately is wrong :-(, and I'm on too deep water to understand how to
fix this at the moment.
N.B: with 00777 flags on the /mnt/gluster/test directory
I have not been able to trigger any unreadable directories
/Anders
>
> Thanks,
> Niels
>
>>
>> Shyam
>>
>> [1] http://review.gluster.org/#/c/6983/
>>
>> ----- Original Message -----
>>> From: "Anders Blomdell" <anders.blomdell@xxxxxxxxxxxxxx>
>>> To: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>
>>> Sent: Tuesday, June 17, 2014 10:53:52 AM
>>> Subject: 3.5.1-beta2 Problems with suid and sgid bits on directories
>>>
> With a glusterfs-3.5.1-0.3.beta2.fc20.x86_64 with a reverted
> 3dc56cbd16b1074d7ca1a4fe4c5bf44400eb63ff (due to local lack of IPv4
> addresses), I get
> weird behavior if I:
>
> 1. Create a directory with suid/sgid/sticky bit set (/mnt/gluster/test)
> 2. Make a subdirectory of #1 (/mnt/gluster/test/dir1)
> 3. Do an add-brick
>
> Before add-brick
>
> 755 /mnt/gluster
> 7775 /mnt/gluster/test
> 2755 /mnt/gluster/test/dir1
>
> After add-brick
>
> 755 /mnt/gluster
> 1775 /mnt/gluster/test
> 755 /mnt/gluster/test/dir1
>
> On the server it looks like this:
>
> 7775 /data/disk1/gluster/test
> 2755 /data/disk1/gluster/test/dir1
> 1775 /data/disk2/gluster/test
> 755 /data/disk2/gluster/test/dir1
>
> Filed as bug:
>
> https://bugzilla.redhat.com/show_bug.cgi?id=1110262
>
> If somebody can point me to where the logic of add-brick is placed, I can
> give
> it a shot (a find/grep on mkdir didn't immediately point me to the right
> place).
>
>
> Regards
>
> Anders Blomdell
>
>
>
>
>>> _______________________________________________
>>> Gluster-devel mailing list
>>> Gluster-devel@xxxxxxxxxxx
>>> http://supercolony.gluster.org/mailman/listinfo/gluster-devel
>>>
>> _______________________________________________
>> Gluster-devel mailing list
>> Gluster-devel@xxxxxxxxxxx
>> http://supercolony.gluster.org/mailman/listinfo/gluster-devel
- --
Anders Blomdell Email: anders.blomdell@xxxxxxxxxxxxxx
Department of Automatic Control
Lund University Phone: +46 46 222 4625
P.O. Box 118 Fax: +46 46 138118
SE-221 00 Lund, Sweden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQEcBAEBAgAGBQJTqTJQAAoJENZYyvaDG8Nc590H/25klV6DD96A8LXPa8Z7UhdG
nHLrMCIe2sOM6DhXrg/lt9QEi5iHpmBQoDk/F+yQ/AWx5c9OWYnDcPYS+4DB0lRp
WZDEdL3DiLYjDCePsibvULVeXrt60nobWo01LMbLrVYh6mXuKee+HwPskCWLtWuo
9CipjyrDUio2ofEhLBNrTJqJ0BUjvSzBOIswcizpeU6oUGllT0GRkXydaaZj4/Gq
IKgCjrSdjS8iy+Rk0yd+LERUakBr8J1PRtuVuYm07MXnaDlDW45P2HhWQUSGciTE
/ujVY7NTCdBO75MlFHgkxNy33tvj3MuSJ53MrGSnLDeepd8oO7eIDuQxGmgk4Zc=
=9tOZ
-----END PGP SIGNATURE-----
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel
