summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVijay Bellur <vbellur@redhat.com>2014-06-23 22:59:14 +0530
committerRaghavendra G <rgowdapp@redhat.com>2014-12-25 23:32:11 -0800
commit79a5b2b991037cc1db5c71f7aa2a32ce712a22e9 (patch)
tree4b436e714f8da8685f9257ed51678389a7622669
parent92cc5d4f64d55675b45b85f612f513500d4e4fc7 (diff)
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 <vbellur@redhat.com> Reviewed-on: http://review.gluster.org/8168 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Raghavendra G <rgowdapp@redhat.com> Tested-by: Raghavendra G <rgowdapp@redhat.com>
-rw-r--r--xlators/features/marker/src/marker-quota.c7
-rw-r--r--xlators/features/marker/src/marker.c32
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 : "<nul>"));
@@ -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;