diff options
author | Sunil Kumar Acharya <sheggodu@redhat.com> | 2017-02-09 16:34:54 +0530 |
---|---|---|
committer | Xavier Hernandez <xhernandez@datalab.es> | 2017-02-15 02:22:27 -0500 |
commit | a6ece44d0bbdecf38252f477b4aa98a76771a328 (patch) | |
tree | 2a49c45c9a1ef6c1532029c30bbe726252050a5b /xlators | |
parent | a88ae92de190af0956013780939ba6bdfd509ff8 (diff) |
cluster/ec: Change log level of messages to DEBUG
Heal failed or passed should not be logged as info. These can be
observed from heal info if the heal is happening or not. If we require
to debug a case where heal is not happening, we can set the level to
DEBUG.
Change-Id: I062668eadd145ef809b25e818e6bca1094f54cd6
BUG: 1420619
Signed-off-by: Sunil Kumar Acharya <sheggodu@redhat.com>
Reviewed-on: https://review.gluster.org/16580
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Ashish Pandey <aspandey@redhat.com>
Diffstat (limited to 'xlators')
-rw-r--r-- | xlators/cluster/ec/src/ec-heal.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/xlators/cluster/ec/src/ec-heal.c b/xlators/cluster/ec/src/ec-heal.c index 548649b9a35..06a4dbb5841 100644 --- a/xlators/cluster/ec/src/ec-heal.c +++ b/xlators/cluster/ec/src/ec-heal.c @@ -2334,6 +2334,7 @@ ec_heal_do (xlator_t *this, void *data, loc_t *loc, int32_t partial) gf_boolean_t blocking = _gf_false; gf_boolean_t need_heal = _gf_false; unsigned char *up_subvols = NULL; + char up_bricks[32]; ec = this->private; @@ -2372,15 +2373,16 @@ ec_heal_do (xlator_t *this, void *data, loc_t *loc, int32_t partial) ret = ec_heal_name (frame, ec, loc->parent, (char *)loc->name, participants); if (ret == 0) { - gf_msg (this->name, GF_LOG_INFO, 0, - EC_MSG_HEAL_SUCCESS, "%s: name heal " + gf_msg_debug (this->name, 0, "%s: name heal " "successful on %lX", loc->path, ec_char_array_to_mask (participants, ec->nodes)); } else { - gf_msg (this->name, GF_LOG_INFO, -ret, - EC_MSG_HEAL_FAIL, "%s: name heal " - "failed", loc->path); + gf_msg_debug (this->name, 0, "%s: name heal " + "failed. ret = %d, subvolumes up = %s", + loc->path, ret, + ec_bin(up_bricks, sizeof(up_bricks), ec->xl_up, + ec->nodes)); } } |