d1681e
From cba83bafc87af2b635c24d1be2d7224a01901c73 Mon Sep 17 00:00:00 2001
d1681e
From: N Balachandran <nbalacha@redhat.com>
d1681e
Date: Tue, 20 Feb 2018 20:08:11 +0530
d1681e
Subject: [PATCH 165/180] cluster/dht: Ignore ENODATA from getxattr for posix
d1681e
 acls
d1681e
d1681e
dht_migrate_file no longer prints an error if getxattr for
d1681e
posix acls fails with ENODATA/ENOATTR.
d1681e
d1681e
upstream: https://review.gluster.org/#/c/19603/
d1681e
d1681e
> Change-Id: Id9ecf6852cb5294c1c154b28d609889ea3420e1c
d1681e
> BUG: 1546954
d1681e
> Signed-off-by: N Balachandran <nbalacha@redhat.com>
d1681e
d1681e
Change-Id: Id9ecf6852cb5294c1c154b28d609889ea3420e1c
d1681e
BUG: 1546945
d1681e
Signed-off-by: N Balachandran <nbalacha@redhat.com>
d1681e
Reviewed-on: https://code.engineering.redhat.com/gerrit/130975
d1681e
Tested-by: RHGS Build Bot <nigelb@redhat.com>
d1681e
Reviewed-by: Amar Tumballi <amarts@redhat.com>
d1681e
---
d1681e
 xlators/cluster/dht/src/dht-rebalance.c | 15 ++++++++-------
d1681e
 1 file changed, 8 insertions(+), 7 deletions(-)
d1681e
d1681e
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c
d1681e
index e620005..9770359 100644
d1681e
--- a/xlators/cluster/dht/src/dht-rebalance.c
d1681e
+++ b/xlators/cluster/dht/src/dht-rebalance.c
d1681e
@@ -2029,13 +2029,14 @@ dht_migrate_file (xlator_t *this, loc_t *loc, xlator_t *from, xlator_t *to,
d1681e
         ret = syncop_getxattr (from, loc, &xattr, POSIX_ACL_ACCESS_XATTR,
d1681e
                                NULL, NULL);
d1681e
         if (ret < 0) {
d1681e
-                gf_msg (this->name, GF_LOG_WARNING, -ret,
d1681e
-                        DHT_MSG_MIGRATE_FILE_FAILED,
d1681e
-                        "Migrate file failed:"
d1681e
-                        "%s: failed to get xattr from %s",
d1681e
-                        loc->path, from->name);
d1681e
-                *fop_errno = -ret;
d1681e
-                ret = -1;
d1681e
+                if ((-ret != ENODATA) && (-ret != ENOATTR)) {
d1681e
+                        gf_msg (this->name, GF_LOG_WARNING, -ret,
d1681e
+                                DHT_MSG_MIGRATE_FILE_FAILED,
d1681e
+                                "Migrate file failed:"
d1681e
+                                "%s: failed to get xattr from %s",
d1681e
+                                loc->path, from->name);
d1681e
+                        *fop_errno = -ret;
d1681e
+                }
d1681e
         } else {
d1681e
                 ret = syncop_setxattr (to, loc, xattr, 0, NULL, NULL);
d1681e
                 if (ret < 0) {
d1681e
-- 
d1681e
1.8.3.1
d1681e