Bug 1752429 - Ctime: Cannot see the "trusted.glusterfs.mdata" xattr for directory on a new brick after rebalance
Summary: Ctime: Cannot see the "trusted.glusterfs.mdata" xattr for directory on a new ...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: ctime
Version: 7
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
Assignee: Kotresh HR
QA Contact:
URL:
Whiteboard:
Depends On: 1728673 1734026
Blocks: 1696809 1752413
TreeView+ depends on / blocked
 
Reported: 2019-09-16 10:52 UTC by Kotresh HR
Modified: 2019-09-26 10:55 UTC (History)
11 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1734026
Environment:
Last Closed: 2019-09-25 04:00:05 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 23431 0 None Merged ctime/rebalance: Heal ctime xattr on directory during rebalance 2019-09-25 04:00:03 UTC

Description Kotresh HR 2019-09-16 10:52:10 UTC
Description of problem:
The "trusted.glusterfs.mdata" xattr was present on a dir on brick1 when it was created, but when added new brick (brick2) to the volume, and did rebalance as well, the xattr was not present on the dir on new brick(brick2).

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

How reproducible:
Always

Steps to Reproduce :
===================
1.Create a 1 brick distributed volume.
2.Mount the volume on a client node using fuse.
3.Create a dir on client node
  #mkdir dir1
4.Check all the internal xattrs for the dir1 on the back-end using:
  #getfattr -de hex -m . /bricks/brick1/dir1
5.Add a new brick to the volume.
6.Trigger rebalance on the volume and let it complete.
7.Check all the internal xattrs for the dir1 on the back-end (old_brick and new_brick) using:
  #getfattr -de hex -m . /bricks/brick1/dir1


Actual Results:
==============
[root@dhcp46-63 ~]# getfattr -de hex -m . /bricks/brick8/vol6-b1/dir1    <------ Old brick
getfattr: Removing leading '/' from absolute path names
# file: bricks/brick8/vol6-b1/dir1
security.selinux=0x73797374656d5f753a6f626a6563745f723a676c7573746572645f627269636b5f743a733000
trusted.gfid=0xeb91e43ca84147ef872a31b119e7c1d4
trusted.glusterfs.dht=0xe92e902a000000007fffffffffffffff
trusted.glusterfs.dht.mds=0x00000000
trusted.glusterfs.mdata=0x010000000000000000000000005d25d1bd0000000021374da7000000005d25d1bd0000000021374da7000000005d25d1bd0000000021374da7



[root@dhcp47-188 dir1]# getfattr -de hex -m . /bricks/brick8/vol6-b2/dir1   <----- New brick
getfattr: Removing leading '/' from absolute path names
# file: bricks/brick8/vol6-b2/dir1
security.selinux=0x73797374656d5f753a6f626a6563745f723a676c7573746572645f627269636b5f743a733000
trusted.gfid=0xeb91e43ca84147ef872a31b119e7c1d4
trusted.glusterfs.dht=0xe92e902a00000000000000007ffffffe



Expected Results:
=================

As the trusted.glusterfs.mdata is an internal xattr, it should also behave like other xattrs (e.g. trusted.glusterfs.dht) and be present on dir1 on new brick as well.


Rebalance status output:
=======================

[root@dhcp47-88 ~]# gluster v rebal vol6 status
                                    Node Rebalanced-files          size       scanned      failures       skipped               status  run time in h:m:s
                               ---------      -----------   -----------   -----------   -----------   -----------         ------------     --------------
                            10.70.47.188                0        0Bytes             0             0             0            completed        0:00:00
                             10.70.46.63                0        0Bytes             0             0             0            completed        0:00:00
volume rebalance: vol6: success

Comment 1 Worker Ant 2019-09-16 10:56:44 UTC
REVIEW: https://review.gluster.org/23431 (ctime/rebalance: Heal ctime xattr on directory during rebalance) posted (#1) for review on release-7 by Kotresh HR

Comment 2 Worker Ant 2019-09-25 04:00:05 UTC
REVIEW: https://review.gluster.org/23431 (ctime/rebalance: Heal ctime xattr on directory during rebalance) merged (#1) on release-7 by Kotresh HR


Note You need to log in before you can comment on or make changes to this bug.