Bug 1721435

Summary:	DHT: Internal xattrs visible on the mount
Product:	[Community] GlusterFS	Reporter:	Nithya Balachandran <nbalacha>
Component:	distribute	Assignee:	Nithya Balachandran <nbalacha>
Status:	CLOSED NEXTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-06-19 06:28:13 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Nithya Balachandran 2019-06-18 09:59:48 UTC

This bug was initially created as a copy of Bug #1721357

Description of problem:

The new pass-thru feature which is enabled by default in master causes internal dht xattrs to be visible on the mount point.


Version-Release number of selected component (if applicable):


How reproducible:

Consistently 
Steps to Reproduce:
1. Install release 3.12 and create a 1x3 volume 
2. Fuse mount the volume and create some directories (dir-1 to dir-10)  and files on the volume
3. Upgrade the node to the latest master and fuse mount the volume
4. Run getfattr -e hex -m . -d /mnt/fuse/dir*



Actual results:
# file: dir-10
security.selinux=0x73797374656d5f753a6f626a6563745f723a6675736566735f743a733000
trusted.glusterfs.dht.mds=0x00000000
user.dirtest=0x64697274657374

# file: dir-2
security.selinux=0x73797374656d5f753a6f626a6563745f723a6675736566735f743a733000
trusted.glusterfs.dht.mds=0x00000000
user.dirtest=0x64697274657374

# file: dir-3
security.selinux=0x73797374656d5f753a6f626a6563745f723a6675736566735f743a733000
trusted.glusterfs.dht.mds=0x00000000
user.dirtest=0x64697274657374

...


Expected results:

trusted.glusterfs.dht.mds is an internal xattr and should not be visible on the mount point.

Additional info:

This was introduced by the pass-through feature.

Comment 1 Worker Ant 2019-06-18 10:26:40 UTC

REVIEW: https://review.gluster.org/22889 (cluster/dht: Strip out dht xattrs) posted (#1) for review on master by N Balachandran

Comment 2 Worker Ant 2019-06-19 06:28:13 UTC

REVIEW: https://review.gluster.org/22889 (cluster/dht: Strip out dht xattrs) merged (#2) on master by Susant Palai