diff options
| author | Sunil Kumar Acharya <sheggodu@redhat.com> | 2017-02-09 16:34:54 +0530 |
|---|---|---|
| committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2017-02-16 09:46:22 -0500 |
| commit | 54848213e366d92338ad5941ab0e6081ec01f6ef (patch) | |
| tree | 7c93085a4d82959331fac3f0ce84e64d168e3134 | |
| parent | 06ab8673f5b0ebe37d358bb6d46bc58fda4a5c83 (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>
Change-Id: I96517f9cf39340ae6883115e464b62339577fc5d
BUG: 1422766
Signed-off-by: Sunil Kumar Acharya <sheggodu@redhat.com>
Reviewed-on: https://review.gluster.org/16635
Reviewed-by: Xavier Hernandez <xhernandez@datalab.es>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Smoke: Gluster Build System <jenkins@build.gluster.org>
| -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)); } } |