Description of problem: For snapshot design it was decided that frequent operations like 'cp -a <large-dir>' should not pause while taking snapshots. This can lead to metadata/xattr mismatches without any afr changelogs because those fops are not going to be blocked while taking snapshots on different bricks. Afr at the time of lookup detects metadata mismatches in stat structures and initiates a self-heal. But if there are any xattr mismatches between two replicas then there is no capability at the moment to detect and heal them. This enhancement tracks the changes required to achieve this change. For afr to detect there are mismatches in xattrs, it should be able to query i.e. listxattr in lookup itself, to avoid extra over the network listxattr call latencies. This requires changes in posix xlator. Afr should use this feature in posix to 'listxattr' and see if there are any mismatches and trigger self-heals on that file probably at the time of fresh-lookup. This behaviour can be exposed in volumes that are exported as part of snapshot. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
REVIEW: http://review.gluster.org/7293 (storage/posix: add list xattr capability to lookup) posted (#1) for review on master by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7293 (storage/posix: add list xattr capability to lookup) posted (#2) for review on master by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7434 (storage/posix: add list xattr capability to lookup) posted (#1) for review on release-3.5 by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7434 (storage/posix: add list xattr capability to lookup) posted (#2) for review on release-3.5 by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7444 (cluster/afr: perform list-xattr during lookup) posted (#1) for review on release-3.5 by Ravishankar N (ravishankar)
REVIEW: http://review.gluster.org/7434 (storage/posix: add list xattr capability to lookup) posted (#3) for review on release-3.5 by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7293 (storage/posix: add list xattr capability to lookup) posted (#3) for review on master by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7444 (cluster/afr: perform list-xattr during lookup) posted (#2) for review on release-3.5 by Ravishankar N (ravishankar)
COMMIT: http://review.gluster.org/7434 committed in release-3.5 by Vijay Bellur (vbellur) ------ commit e49e56c5e81e7e6a4cc4d0ac2982446dd1887ef9 Author: Pranith Kumar K <pkarampu> Date: Wed Mar 19 07:52:48 2014 +0530 storage/posix: add list xattr capability to lookup BUG: 1078061 Change-Id: Ib84f40c84e2d66c86b1cec4a7cceb5aea189ba4a Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/7434 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Xavier Hernandez <xhernandez> Reviewed-by: Ravishankar N <ravishankar> Reviewed-by: Vijay Bellur <vbellur>
COMMIT: http://review.gluster.org/7444 committed in release-3.5 by Vijay Bellur (vbellur) ------ commit 0d2dbd569f5ee187fc7b864c30cc668fee465bb1 Author: Ravishankar N <ravishankar> Date: Fri Apr 11 05:45:56 2014 +0000 cluster/afr: perform list-xattr during lookup Detect and heal mismatching user extended attributes during lookup. Depends on: http://review.gluster.org/#/c/7434/ Change-Id: I49410aafd319ac159fdf9e6f9201871bbf2f67bd BUG: 1078061 Signed-off-by: Ravishankar N <ravishankar> Reviewed-on: http://review.gluster.org/7444 Reviewed-by: Pranith Kumar Karampuri <pkarampu> Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Vijay Bellur <vbellur>
REVIEW: http://review.gluster.org/7293 (storage/posix: add list xattr capability to lookup) posted (#4) for review on master by Pranith Kumar Karampuri (pkarampu)
COMMIT: http://review.gluster.org/7293 committed in master by Anand Avati (avati) ------ commit 91ab65f812ec3b674f53230eacbd0d71964d3d01 Author: Pranith Kumar K <pkarampu> Date: Wed Mar 19 07:52:48 2014 +0530 storage/posix: add list xattr capability to lookup BUG: 1078061 Change-Id: Ie26d28b8a74aa0d1eceff14a84c3cd3e302dcdb5 Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/7293 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Anand Avati <avati>
The first (and last?) Beta for GlusterFS 3.5.1 has been released [1]. Please verify if the release solves this bug report for you. In case the glusterfs-3.5.1beta release does not have a resolution for this issue, leave a comment in this bug and move the status to ASSIGNED. If this release fixes the problem for you, leave a note and change the status to VERIFIED. Packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an "updates-testing" repository) infrastructure for your distribution. [1] http://supercolony.gluster.org/pipermail/gluster-users/2014-May/040377.html [2] http://supercolony.gluster.org/pipermail/gluster-users/
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.5.1, please reopen this bug report. glusterfs-3.5.1 has been announced on the Gluster Users mailinglist [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://supercolony.gluster.org/pipermail/gluster-users/2014-June/040723.html [2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user