diff options
| author | Susant Palai <spalai@redhat.com> | 2019-07-16 10:31:46 +0530 |
|---|---|---|
| committer | N Balachandran <nbalacha@redhat.com> | 2019-07-18 10:19:17 +0000 |
| commit | 41f72ac9453c24bea85709203948c41c5772edec (patch) | |
| tree | 3cba3693ef3ceabe227c6c7a427e046c3fc13e24 | |
| parent | 477bbe36dc2a05e00ffb137450fa18a0aba8bf6a (diff) | |
dht: log getxattr failure for node-uuid at "DEBUG"
There are two ways to fetch node-uuid information from dht.
1 - #define GF_XATTR_LIST_NODE_UUIDS_KEY "trusted.glusterfs.list-node-uuids"
This key is used by AFR.
2 - #define GF_REBAL_FIND_LOCAL_SUBVOL "glusterfs.find-local-subvol"
This key is used for non-afr volume type.
We do two getxattr operations. First on the #1 key followed by on #2 if
getxattr on #1 key fails.
Since the parent function "dht_init_local_subvols_and_nodeuuids" logs failure,
moving the log-level to DEBUG in dht_find_local_subvol_cbk.
fixes: bz#1730175
Change-Id: I4d88244dc26587b111ca5b00d4c00118efdaac14
Signed-off-by: Susant Palai <spalai@redhat.com>
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index 61e0318473a..8a1ec2baa7a 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -4239,8 +4239,11 @@ dht_find_local_subvol_cbk(call_frame_t *frame, void *cookie, xlator_t *this, local->op_ret = -1; local->op_errno = op_errno; UNLOCK(&frame->lock); - gf_msg(this->name, GF_LOG_ERROR, op_errno, DHT_MSG_GET_XATTR_FAILED, - "getxattr err for dir"); + if (op_errno == ENODATA) + gf_msg_debug(this->name, 0, "failed to get node-uuid"); + else + gf_msg(this->name, GF_LOG_ERROR, op_errno, + DHT_MSG_GET_XATTR_FAILED, "failed to get node-uuid"); goto post_unlock; } |