diff options
| author | Pranith Kumar K <pranithk@gluster.com> | 2010-07-27 03:13:55 +0000 |
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2010-07-27 06:48:47 -0700 |
| commit | 232235a9ddeaf33159ee8af809efee440b06c6d1 (patch) | |
| tree | 093c89fc1398656da2ab7cdae192669ccc624eea /xlators/cluster/afr/src/afr-self-heal-algorithm.c | |
| parent | 7c3ee2fd042c56ef6f53159f1ae9dd9a8f91c6f4 (diff) | |
cluster/afr: Logging improvement for self-heal
Signed-off-by: Pranith Kumar K <pranithk@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 1037 (selfheal information in normal logging mode)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1037
Diffstat (limited to 'xlators/cluster/afr/src/afr-self-heal-algorithm.c')
| -rw-r--r-- | xlators/cluster/afr/src/afr-self-heal-algorithm.c | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-algorithm.c b/xlators/cluster/afr/src/afr-self-heal-algorithm.c index daac0ce476c..51f5fd3df5b 100644 --- a/xlators/cluster/afr/src/afr-self-heal-algorithm.c +++ b/xlators/cluster/afr/src/afr-self-heal-algorithm.c @@ -947,17 +947,20 @@ sh_diff_loop_driver (call_frame_t *frame, xlator_t *this) int recurse = 0; off_t offset = 0; + char sh_type_str[256] = {0,}; priv = this->private; local = frame->local; sh = &local->self_heal; sh_priv = sh->private; + afr_self_heal_type_str_get(sh, sh_type_str, sizeof(sh_type_str)); + if (sh->op_failed) { if (sh_priv->loops_running == 0) { - gf_log (this->name, GF_LOG_TRACE, - "diff self-heal aborting on %s", - local->loc.path); + gf_log (this->name, GF_LOG_ERROR, + "diff %s self-heal aborting on %s", + sh_type_str, local->loc.path); sh_diff_private_cleanup (frame, this); local->self_heal.algo_abort_cbk (frame, this); @@ -969,14 +972,14 @@ sh_diff_loop_driver (call_frame_t *frame, xlator_t *this) if (sh_priv->offset >= sh->file_size) { if (sh_priv->loops_running == 0) { gf_log (this->name, GF_LOG_TRACE, - "diff self-heal completed on %s", - local->loc.path); + "diff %s self-heal completed on %s", + sh_type_str, local->loc.path); - gf_log (this->name, GF_LOG_DEBUG, - "diff self-heal on %s: %d blocks of %d were different (%.2f%%)", - local->loc.path, sh_priv->diff_blocks, - sh_priv->total_blocks, + gf_log (this->name, GF_LOG_NORMAL, + "diff %s self-heal on %s: %d blocks of %d were different (%.2f%%)", + sh_type_str, local->loc.path, + sh_priv->diff_blocks, sh_priv->total_blocks, ((sh_priv->diff_blocks * 1.0)/sh_priv->total_blocks) * 100); sh_diff_private_cleanup (frame, this); |