From 79a5b2b991037cc1db5c71f7aa2a32ce712a22e9 Mon Sep 17 00:00:00 2001 From: Vijay Bellur Date: Mon, 23 Jun 2014 22:59:14 +0530 Subject: features/marker: log message clean up 1. Changed log messages to be more appropriate. 2. Changed loglevel of failures in fop_cbks to be recorded as TRACE. Logging of failures at higher loglevels is unessential in non-endpoint translators. 3. Removed a log message related to memory allocation failure. BUG: 1174087 Change-Id: I63c560c3bbd12706357fb3f696378c1a1e1efb44 Signed-off-by: Vijay Bellur Reviewed-on: http://review.gluster.org/8168 Tested-by: Gluster Build System Reviewed-by: Kaleb KEITHLEY Reviewed-by: Raghavendra G Tested-by: Raghavendra G --- xlators/features/marker/src/marker-quota.c | 7 +++---- xlators/features/marker/src/marker.c | 32 ++++++++++++++---------------- 2 files changed, 18 insertions(+), 21 deletions(-) diff --git a/xlators/features/marker/src/marker-quota.c b/xlators/features/marker/src/marker-quota.c index 37c5cfa9225..b8d7a774377 100644 --- a/xlators/features/marker/src/marker-quota.c +++ b/xlators/features/marker/src/marker-quota.c @@ -557,7 +557,7 @@ mq_readdir_cbk (call_frame_t *frame, local = mq_local_ref (frame->local); if (op_ret == -1) { - gf_log (this->name, GF_LOG_DEBUG, + gf_log (this->name, GF_LOG_TRACE, "readdir failed %s", strerror (op_errno)); local->err = -1; @@ -1295,9 +1295,8 @@ mq_get_parent_inode_local (xlator_t *this, quota_local_t *local) local->ctx = ctx; if (list_empty (&ctx->contribution_head)) { - gf_log_callingfn (this->name, GF_LOG_WARNING, - "contribution node list is empty which " - "is an error"); + gf_log_callingfn (this->name, GF_LOG_ERROR, + "contribution node list is empty"); ret = -1; goto out; } diff --git a/xlators/features/marker/src/marker.c b/xlators/features/marker/src/marker.c index cec9dff05a3..dcbbbd49190 100644 --- a/xlators/features/marker/src/marker.c +++ b/xlators/features/marker/src/marker.c @@ -588,7 +588,7 @@ marker_mkdir_cbk (call_frame_t *frame, void *cookie, xlator_t *this, if (op_ret == -1) { gf_log (this->name, GF_LOG_TRACE, "error occurred " - "while Creating a file %s", strerror (op_errno)); + "while creating directory %s", strerror (op_errno)); } local = (marker_local_t *) frame->local; @@ -662,7 +662,7 @@ marker_create_cbk (call_frame_t *frame, void *cookie, xlator_t *this, if (op_ret == -1) { gf_log (this->name, GF_LOG_TRACE, "error occurred " - "while Creating a file %s", strerror (op_errno)); + "while creating file %s", strerror (op_errno)); } local = (marker_local_t *) frame->local; @@ -1818,8 +1818,8 @@ marker_mknod_cbk (call_frame_t *frame, void *cookie, xlator_t *this, marker_conf_t *priv = NULL; if (op_ret == -1) { - gf_log (this->name, GF_LOG_TRACE, "%s occurred while " - "creating symlinks ", strerror (op_errno)); + gf_log (this->name, GF_LOG_TRACE, "%s occurred with " + "mknod ", strerror (op_errno)); } local = (marker_local_t *) frame->local; @@ -2358,8 +2358,8 @@ marker_fsetxattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this, marker_conf_t *priv = NULL; if (op_ret == -1) { - gf_log (this->name, GF_LOG_TRACE, "%s occurred while " - "creating symlinks ", strerror (op_errno)); + gf_log (this->name, GF_LOG_TRACE, "%s occurred in " + "fsetxattr", strerror (op_errno)); } local = (marker_local_t *) frame->local; @@ -2426,8 +2426,8 @@ marker_fsetattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this, marker_conf_t *priv = NULL; if (op_ret == -1) { - gf_log (this->name, GF_LOG_ERROR, "%s occurred while " - "creating symlinks ", strerror (op_errno)); + gf_log (this->name, GF_LOG_TRACE, "%s occurred in " + "fsetattr ", strerror (op_errno)); } local = (marker_local_t *) frame->local; @@ -2496,8 +2496,7 @@ marker_setattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this, frame->local = NULL; if (op_ret == -1) { - gf_log (this->name, ((op_errno == ENOENT) ? GF_LOG_DEBUG : - GF_LOG_ERROR), + gf_log (this->name, GF_LOG_TRACE, "%s occurred during setattr of %s", strerror (op_errno), (local ? local->loc.path : "")); @@ -2559,8 +2558,10 @@ marker_removexattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this, marker_conf_t *priv = NULL; if (op_ret == -1) { - gf_log (this->name, GF_LOG_ERROR, "%s occurred while " - "creating symlinks ", strerror (op_errno)); + gf_log (this->name, GF_LOG_TRACE, + "%s occurred while " + "removing extended attribute", + strerror (op_errno)); } local = (marker_local_t *) frame->local; @@ -2869,7 +2870,7 @@ mem_acct_init (xlator_t *this) if (ret != 0) { gf_log(this->name, GF_LOG_ERROR, "Memory accounting init" - "failed"); + " failed"); return ret; } @@ -2906,14 +2907,11 @@ init_xtime_priv (xlator_t *this, dict_t *options) if (ret == -1){ priv->marker_xattr = NULL; - - gf_log (this->name, GF_LOG_ERROR, - "Failed to allocate memory"); goto out; } gf_log (this->name, GF_LOG_DEBUG, - "the volume-uuid = %s", priv->volume_uuid); + "volume-uuid = %s", priv->volume_uuid); } else { priv->volume_uuid = NULL; -- cgit